Page MenuHomePhabricator
Differential D6901

[keyserver] Introduce `getOLMUtility(...)` and initialize OLM on `keyserver` start
ClosedPublic
Actions

Authored by atul on Feb 27 2023, 1:57 PM.
Tags
None
Referenced Files
F3602789: D6901.id.diff
Tue, Dec 31, 9:18 AM
Unknown Object (File)
Sun, Dec 29, 7:07 PM
Unknown Object (File)
Sun, Dec 29, 7:07 PM
Unknown Object (File)
Sun, Dec 29, 7:07 PM
Unknown Object (File)
Sun, Dec 29, 7:07 PM
Unknown Object (File)
Sun, Dec 29, 7:04 PM
Unknown Object (File)
Mon, Dec 23, 2:31 AM
Unknown Object (File)
Sun, Dec 22, 9:47 PM
View All Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMM06732ac6a7f2: [keyserver] Introduce `getOLMUtility(...)` and initialize OLM on `keyserver`…
Summary

Initialize OLM by calling await olm.init() on keyserver start and introduce getOLMUtility to potentially access cached OLMUtility... or instantiate a new one.

Test Plan
  • Logged in a bunch of times and ensured things worked as expected.
  • Set breakpoint on cache hit:

8fe3ee.png (632×1 px, 147 KB)

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Feb 27 2023, 1:57 PM
atul published this revision for review.Feb 27 2023, 1:58 PM
atul mentioned this in D6897: [keyserver] Verify `signedIdentityKeysBlob` signature in `logInResponder`.
atul updated this revision to Diff 23179.Feb 27 2023, 1:59 PM

annotate

atul edited the test plan for this revision. (Show Details)Feb 27 2023, 1:59 PM
Harbormaster completed remote builds in B16957: Diff 23178.Feb 27 2023, 2:13 PM
Harbormaster completed remote builds in B16958: Diff 23179.
ashoat accepted this revision.Feb 27 2023, 2:35 PM
This revision is now accepted and ready to land.Feb 27 2023, 2:35 PM
atul updated this revision to Diff 23188.Feb 27 2023, 3:20 PM

rebase + land

This revision was landed with ongoing or failed builds.Feb 27 2023, 3:20 PM
Closed by commit rCOMM06732ac6a7f2: [keyserver] Introduce `getOLMUtility(...)` and initialize OLM on `keyserver`… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM06732ac6a7f2: [keyserver] Introduce `getOLMUtility(...)` and initialize OLM on `keyserver`….
Harbormaster completed remote builds in B16965: Diff 23188.Feb 27 2023, 3:34 PM
ashoat mentioned this in D6923: [keyserver] Tunnelbroker integration - Adding Tunnelbroker publisher and connector.Mar 2 2023, 7:53 AM

Revision Contents
Changeset List

PathSize
keyserver/
src/
2 lines
responders/
6 lines
utils/
12 lines

Diff 23179

View Options

keyserver/src/keyserver.js

Loading...
View Options

keyserver/src/responders/user-responders.js

Loading...
View Options

keyserver/src/utils/olm-utils.js

Loading...