Page MenuHomePhabricator
Differential D12669

Implement peer notifications session creation as outbound on web and add notifs outbound session creator to OlmAPI
ClosedPublic
Actions

Authored by marcin on Jul 4 2024, 9:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 23, 7:03 AM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
Unknown Object (File)
Sun, Dec 22, 5:32 PM
View All Files
Subscribers
ashoat
kamil

Details

Reviewers
bartek
tomek
inka
kamil
Commits
rCOMM33730e29f317: Implement peer notifications session creation as outbound on web and add notifs…
Summary

This differential implements outbound notifications session creation on web. It also adds this functionality to OlmAPI so that code in lib can create notifs sessions

Test Plan

Tested in D12673

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin created this revision.Jul 4 2024, 9:23 AM
Herald added a subscriber: ashoat. · View Herald TranscriptJul 4 2024, 9:23 AM
marcin added a parent revision: D12668: Implement peer notifications session creation as outbound on native.Jul 4 2024, 9:24 AM
marcin added a child revision: D12670: Implement notification encryption on native.
Harbormaster completed remote builds in B30148: Diff 42039.Jul 4 2024, 9:39 AM
marcin requested review of this revision.Jul 4 2024, 9:39 AM
marcin edited the test plan for this revision. (Show Details)Jul 4 2024, 9:44 AM
tomek accepted this revision.Jul 8 2024, 4:12 AM
tomek added inline comments.
lib/types/crypto-types.js
178 ↗(On Diff #42039)

Why do we need the content keys?

web/push-notif/notif-crypto-utils.js
441 ↗(On Diff #42039)

Should we mention something about notifs in this name?

This revision is now accepted and ready to land.Jul 8 2024, 4:12 AM
marcin added inline comments.Jul 12 2024, 2:22 AM
lib/types/crypto-types.js
178 ↗(On Diff #42039)

We need deviceID and deviceID is the ed25519 part of contentInboundKeys.

web/push-notif/notif-crypto-utils.js
441 ↗(On Diff #42039)

I don't think it is necessary since we only store notifs data in IndexedDB.

kamil accepted this revision.Jul 16 2024, 4:32 AM
kamil added a subscriber: kamil.
kamil added inline comments.
lib/types/crypto-types.js
170–180 ↗(On Diff #42039)

It's a bit unclear what are differences between these two and which one to use when needed - can we somehow improve naming?

178 ↗(On Diff #42039)

that being said I think it's better to just pass deviceID, but up to you

marcin updated this revision to Diff 42727.Jul 24 2024, 5:45 AM

Address review

Harbormaster failed remote builds in B30631: Diff 42727!Jul 24 2024, 5:57 AM
marcin updated this revision to Diff 42935.Jul 30 2024, 3:22 AM

Update phab

Harbormaster completed remote builds in B30790: Diff 42935.Jul 30 2024, 3:45 AM
marcin updated this revision to Diff 43072.Aug 5 2024, 4:05 AM

Rebase before landing

Harbormaster completed remote builds in B30892: Diff 43072.Aug 5 2024, 4:34 AM
Closed by commit rCOMM33730e29f317: Implement peer notifications session creation as outbound on web and add notifs… (authored by marcin, committed by Marcin Wąsowicz <marcinwasowicz@MacBook-Pro-Marcin.local>). · Explain WhyAug 5 2024, 6:39 AM
This revision was automatically updated to reflect the committed changes.
Marcin Wąsowicz <marcinwasowicz@MacBook-Pro-Marcin.local> added a commit: rCOMM33730e29f317: Implement peer notifications session creation as outbound on web and add notifs….

Revision Contents
Changeset List

PathSize
lib/
keyserver-conn/
2 lines
shared/
2 lines
types/
7 lines
utils/
__mocks__/
3 lines
native/
crypto/
21 lines
web/
crypto/
7 lines
push-notif/
19 lines
shared-worker/
worker/
167 lines

Diff 42935

View Options

lib/keyserver-conn/keyserver-auth.js

Loading...
View Options

lib/shared/crypto-utils.js

Loading...
View Options

lib/types/crypto-types.js

Loading...
View Options

lib/utils/__mocks__/config.js

Loading...
View Options

native/crypto/olm-api.js

Loading...
View Options

web/crypto/olm-api.js

Loading...
View Options

web/push-notif/notif-crypto-utils.js

Loading...
View Options

web/shared-worker/worker/worker-crypto.js

Loading...