Page MenuHomePhabricator

Implement new endpoint on the keyserver to create notifs olm session on client demand
AcceptedPublic

Authored by marcin on Aug 15 2024, 10:22 AM.
Tags
None
Referenced Files
F3197225: D13090.id43409.diff
Sat, Nov 9, 7:48 AM
Unknown Object (File)
Fri, Oct 25, 6:43 PM
Unknown Object (File)
Tue, Oct 22, 1:16 PM
Unknown Object (File)
Tue, Oct 22, 10:23 AM
Unknown Object (File)
Tue, Oct 22, 9:15 AM
Unknown Object (File)
Sep 18 2024, 4:35 PM
Unknown Object (File)
Sep 18 2024, 10:05 AM
Unknown Object (File)
Sep 14 2024, 11:47 PM
Subscribers
None

Details

Reviewers
tomek
kamil
ashoat
Summary

This differential implements new endpoint on the keyserver that creates new notifs olm session on client demand.

Test Plan

Tested with final diff in the stack

Diff Detail

Repository
rCOMM Comm
Branch
marcin/eng-9025
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Worth considering potential race conditions with this endpoint and INITIAL_NOTIFICATIONS_ENCRYPTED_MESSAGE to avoid diverged session (keyserver and client session are different),

keyserver/src/endpoints.js
495

I think the validator should be named recreateNotifsOlmSessionInputValidator to match convention and placed in file with responder

lib/actions/keyserver-actions.js
11

I would put this in user-actions.js along with getOlmSessionInitializationData

15–18

I would create a read-only type RecreateNotifsOlmSessionInput and use it instead of spreading in two places

This revision is now accepted and ready to land.Aug 19 2024, 4:28 AM