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
F2027316: D7092.diff
Mon, Jun 17, 8:00 PM
Unknown Object (File)
Sun, Jun 9, 2:12 PM
Unknown Object (File)
Thu, May 30, 5:29 AM
Unknown Object (File)
Tue, May 28, 12:52 AM
Unknown Object (File)
Sat, May 25, 2:14 PM
Unknown Object (File)
Sat, May 25, 2:14 PM
Unknown Object (File)
Sat, May 25, 2:14 PM
Unknown Object (File)
May 6 2024, 12:42 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!