Page MenuHomePhabricator
Differential D11220

[lib][native] Initialize notif Olm channel during keyserver session recovery
ClosedPublic
Actions

Authored by ashoat on Mar 3 2024, 8:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 16, 12:36 PM
Unknown Object (File)
Wed, Apr 16, 5:51 AM
Unknown Object (File)
Wed, Apr 16, 3:54 AM
Unknown Object (File)
Tue, Apr 15, 12:50 AM
Unknown Object (File)
Wed, Apr 9, 12:25 AM
Unknown Object (File)
Tue, Apr 8, 9:07 PM
Unknown Object (File)
Tue, Apr 8, 7:58 PM
Unknown Object (File)
Fri, Mar 21, 2:27 PM
View All 73 Files
Subscribers
None

Details

Reviewers
marcin
inka
tomek
Commits
rCOMMf022de89c4c5: [lib][native] Initialize notif Olm channel during keyserver session recovery
Summary

I missed adding this optional param, which was definitely an omission. Besides passing it in, this diff makes it a required param.

Depends on D11219

Test Plan

In addition to the plan described here: https://gist.github.com/Ashoat/75ab690d5c53cdd68a51b02e03e27c58

I also made sure that after session recovery, than an entry in the olm_sessions MariaDB table existed for the new cookie with is_content = 0

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
components/
15 lines
keyserver-conn/
2 lines
utils/
2 lines
native/
account/
16 lines

Diff 37845

View Options

lib/components/keyserver-connection-handler.js

Loading...
View Options

lib/keyserver-conn/recovery-utils.js

Loading...
View Options

lib/utils/config.js

Loading...
View Options

native/account/legacy-recover-keyserver-session.js

Loading...