Page MenuHomePhabricator

[web-db] not include `crypto`, `fs`, and `path` polyfills for workers
ClosedPublic

Authored by kamil on Mar 7 2023, 8:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 21 2024, 10:59 PM
Unknown Object (File)
Feb 21 2024, 1:56 AM
Unknown Object (File)
Feb 21 2024, 1:56 AM
Unknown Object (File)
Feb 21 2024, 1:56 AM
Unknown Object (File)
Feb 11 2024, 5:24 PM
Unknown Object (File)
Feb 11 2024, 5:24 PM
Unknown Object (File)
Feb 11 2024, 5:22 PM
Unknown Object (File)
Feb 11 2024, 2:10 PM
Subscribers

Details

Summary

Docs: webpack.

Without this change including sql,js module in worker will cause issues with module not found.

Depends on D6987

Test Plan

cd web && yarn dev

Diff Detail

Repository
rCOMM Comm
Branch
db-on-web-2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Mar 7 2023, 10:05 AM

The title is somewhat misleading, but mostly the fault of Webpack docs. Before this diff, the polyfills were not included. After this diff, each of these is set to the empty object (I think)

This revision is now accepted and ready to land.Mar 7 2023, 4:25 PM