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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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