Page MenuHomePhabricator
Differential D13468

Add AES cryptography to EncryptedNotifUtilsAPI
ClosedPublic
Actions

Authored by marcin on Sep 25 2024, 11:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 9:20 AM
Unknown Object (File)
Wed, Dec 18, 9:20 AM
Unknown Object (File)
Wed, Dec 18, 9:20 AM
Unknown Object (File)
Nov 20 2024, 9:59 AM
Unknown Object (File)
Nov 18 2024, 5:57 AM
Unknown Object (File)
Nov 10 2024, 4:17 PM
Unknown Object (File)
Nov 8 2024, 3:02 AM
Unknown Object (File)
Nov 1 2024, 4:22 PM
View All 60 Files
Subscribers
ashoat

Details

Reviewers
kamil
tomek
bartek
Commits
rCOMM0a028741d881: Add AES cryptography to EncryptedNotifUtilsAPI
Summary

This differential adds AES cryptography to EncryptedNotifUtilsAPI

Test Plan

Tested in the final diff

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

marcin created this revision.Sep 25 2024, 11:11 AM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 25 2024, 11:11 AM
marcin added a parent revision: D13467: Enable AES encryption with custom IV on web/keyserver.Sep 25 2024, 11:12 AM
marcin added a child revision: D13469: Create large notifications in thick threads.
Harbormaster completed remote builds in B31851: Diff 44561.Sep 25 2024, 11:28 AM
marcin requested review of this revision.Sep 25 2024, 11:28 AM
marcin removed a parent revision: D13467: Enable AES encryption with custom IV on web/keyserver.Sep 26 2024, 1:30 AM
marcin updated this revision to Diff 44590.Sep 26 2024, 1:31 AM

Update to simpler approach

Harbormaster completed remote builds in B31872: Diff 44590.Sep 26 2024, 1:48 AM
bartek accepted this revision.Sep 26 2024, 2:14 AM
bartek added inline comments.
web/push-notif/encrypted-notif-utils-api.js
50 ↗(On Diff #44590)

TIL: we have Webpack polyfill for Buffer on web.
I was sure Buffer is a Node-only type and I was using plain ArrayBuffer and Uint8Array on web

This revision is now accepted and ready to land.Sep 26 2024, 2:14 AM
marcin edited the test plan for this revision. (Show Details)Sep 27 2024, 7:10 AM
marcin updated this revision to Diff 44641.Sep 27 2024, 7:41 AM

Rebase

This revision was landed with ongoing or failed builds.Sep 27 2024, 7:48 AM
Closed by commit rCOMM0a028741d881: Add AES cryptography to EncryptedNotifUtilsAPI (authored by marcin). · Explain Why
This revision was automatically updated to reflect the committed changes.
marcin added a commit: rCOMM0a028741d881: Add AES cryptography to EncryptedNotifUtilsAPI.
Harbormaster completed remote builds in B31908: Diff 44641.Sep 27 2024, 8:02 AM

Revision Contents
Changeset List

PathSize
keyserver/
src/
push/
15 lines
lib/
types/
6 lines
utils/
__mocks__/
3 lines
native/
push/
19 lines
web/
push-notif/
23 lines

Diff 44644

View Options

keyserver/src/push/encrypted-notif-utils-api.js

Loading...
View Options

lib/types/notif-types.js

Loading...
View Options

lib/utils/__mocks__/config.js

Loading...
View Options

native/push/encrypted-notif-utils-api.js

Loading...
View Options

web/push-notif/encrypted-notif-utils-api.js

Loading...