Updates in-memory CryptoStore to work with session versions.
Depends on D11582
Details
Details
Tested in TBD
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- publish-session-resetting
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| web/shared-worker/worker/worker-crypto.js | ||
|---|---|---|
| 63 ↗ | (On Diff #39043) | Sorry for adding this comment late, but this could be made read-only |
| web/shared-worker/worker/worker-crypto.js | ||
|---|---|---|
| 63 ↗ | (On Diff #39043) | If I recall correctly this was by design, so we can modify one session in in-memory cryptoStore (see line 74) without updating the entire object. If you still think it's better to make this read-only and update rest of the code I can put up a diff |
| web/shared-worker/worker/worker-crypto.js | ||
|---|---|---|
| 63 ↗ | (On Diff #39043) | Thanks for explaining! Looks like it's been thought through, so it's fine to leave as-is |