Page MenuHomePhabricator

Establish olm notifs session on both sides during registration
ClosedPublic

Authored by marcin on Apr 27 2023, 6:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 26 2024, 4:39 AM
Unknown Object (File)
Feb 23 2024, 1:10 AM
Unknown Object (File)
Feb 23 2024, 1:10 AM
Unknown Object (File)
Feb 22 2024, 10:04 PM
Unknown Object (File)
Feb 20 2024, 7:21 AM
Unknown Object (File)
Feb 20 2024, 7:16 AM
Unknown Object (File)
Feb 20 2024, 6:33 AM
Unknown Object (File)
Feb 20 2024, 5:05 AM
Subscribers

Details

Summary

This differential implements olm notifs session establishment during account creation.

Test Plan

Create new account. Ensure new entry in olm_sessions table appears

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat requested changes to this revision.Apr 27 2023, 1:10 PM
ashoat added inline comments.
keyserver/src/creators/account-creator.js
132–138 ↗(On Diff #25828)

See feedback in D7654. Please be more thoughtful when introducing new await keywords

native/account/register-panel.react.js
295–297 ↗(On Diff #25828)

See feedback in D7654. Please be more thoughtful when introducing new await keywords

This revision now requires changes to proceed.Apr 27 2023, 1:10 PM

Better usage of awaits. Ensure cookie iscreated prior to olm session.

This revision is now accepted and ready to land.May 2 2023, 10:13 AM

Rename call to get and rebase before landing

This revision is now accepted and ready to land.May 31 2023, 8:36 AM