Page MenuHomePhabricator

[web] Make identity service client runnable on shared worker
ClosedPublic

Authored by michal on Mar 7 2024, 8:08 AM.
Tags
None
Referenced Files
F3384047: D11273.id38118.diff
Thu, Nov 28, 7:09 PM
Unknown Object (File)
Mon, Nov 25, 3:31 AM
Unknown Object (File)
Mon, Nov 25, 3:27 AM
Unknown Object (File)
Mon, Nov 25, 1:31 AM
Unknown Object (File)
Thu, Nov 7, 3:08 AM
Unknown Object (File)
Oct 15 2024, 7:25 PM
Unknown Object (File)
Oct 15 2024, 7:25 PM
Unknown Object (File)
Oct 15 2024, 7:25 PM
Subscribers

Details

Summary

[ENG-6766 : Move IdentityServiceClientWrapper to shared worker](https://linear.app/comm/issue/ENG-6766/move-identityserviceclientwrapper-to-shared-worker)

Currently IdentityServiceClientWrapper has two things which don't work on the shared worker:

  • getConfig which is registered only in browser window js context and not worker
  • opaque WASM path needs to be different on webworker (because the inner fetch base origin is different)

this diff changes both of these values to constructor arguments instead of using global context.

Depends on D11272

Test Plan

Make sure logging in to the identity service still works.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

michal requested review of this revision.Mar 7 2024, 8:23 AM
kamil added inline comments.
web/grpc/identity-service-client-wrapper.js
45

this seems more like string - not boolean flag

This revision is now accepted and ready to land.Mar 15 2024, 10:14 AM