Page MenuHomePhabricator

[web-db] Use unique filename for each prod `sql-wasm.wasm` asset
ClosedPublic

Authored by kamil on Mar 17 2023, 4:16 AM.
Tags
None
Referenced Files
F578581: D7092.id23820.diff
Thu, Jun 8, 10:49 AM
F578580: D7092.id.diff
Thu, Jun 8, 10:49 AM
Unknown Object (File)
Apr 3 2023, 7:36 PM
Unknown Object (File)
Mar 30 2023, 2:17 AM
Unknown Object (File)
Mar 28 2023, 10:02 AM
Unknown Object (File)
Mar 24 2023, 4:11 PM
Unknown Object (File)
Mar 21 2023, 4:08 PM
Unknown Object (File)
Mar 21 2023, 10:40 AM
Subscribers

Details

Summary

ENG-3329

  • adding contenthash to sql.js asset file
  • adding mafiest plugin
Test Plan

Run in dev and prod modes, check if worker can access and use file

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 17 2023, 4:29 AM
This revision is now accepted and ready to land.Mar 17 2023, 5:52 AM
keyserver/src/responders/website-responders.js
435 ↗(On Diff #23820)

Where do we consume this?

web/webpack.config.cjs
136 ↗(On Diff #23820)

Can you share an example of the contents of this file?

keyserver/src/responders/website-responders.js
435 ↗(On Diff #23820)

Ah I see, D6993 explains

web/webpack.config.cjs
136 ↗(On Diff #23820)

sure:

{
  "pushNotif.js": "pushNotif.build.js",
  "database.js": "database.build.js",
  "sql-wasm.wasm": "sql-wasm.057823fecaed.wasm"
}

(just reminding that this is the file in webworkers directory so it's not overriding manifest.json where olm is defined)

web/webpack.config.cjs
136 ↗(On Diff #23820)

Looks good!