Introduce EncryptedNotifUtilsAPI and implement it on the keyserver
AcceptedPublic
Actions

Authored by marcin on Tue, Jun 11, 10:36 AM.

Details

Reviewers

kamil
tomek

Summary

This differential introduces EncryptedNotifUtilsAPI. It serves the same purpose as olmAPI or sqliteAPI. During notification generation and encryption we will use the same utilities on all platforms but their implementations will be platform specific. This diff implements it for keyserver only for now. Future diffs will implement it on native and web as well.

Test Plan

Flow
Test all notifs for all platforms and large notifs as well.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

marcin created this revision.Tue, Jun 11, 10:36 AM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Jun 11, 10:36 AM

marcin added a parent revision: D12395: Enable encrypted notifications to have keyservevrID or senderDeviceID.Tue, Jun 11, 10:36 AM

marcin added a child revision: D12397: Move notification creaction and encryption to lib except for APNs noifs..Tue, Jun 11, 10:40 AM

Harbormaster completed remote builds in B29604: Diff 41213.Tue, Jun 11, 10:54 AM

marcin requested review of this revision.Tue, Jun 11, 10:54 AM

ashoat added inline comments.Wed, Jun 12, 8:45 AM

lib/types/notif-types.js
2 ↗	(On Diff #41213)	Nit: you seem to have accidentally cut a line break here

I would consider registering config for keyserver to be able to use encryptedNotifUtilsAPI directly - passing it as a param trough all these functions makes it messy. In the future, we'll also want sqliteAPI to be implemented on keyserver, but this is something we can discuss in the Linear follow-up task because right now Config type is designed specifically for web & native.

This revision is now accepted and ready to land.Wed, Jun 19, 3:15 PM

Rebase

Harbormaster completed remote builds in B29820: Diff 41553.Thu, Jun 20, 4:37 AM