Page MenuHomePhabricator

Implement API to encrypt Windows notifications
ClosedPublic

Authored by marcin on Dec 5 2023, 4:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Thu, Oct 3, 10:11 AM
Unknown Object (File)
Aug 29 2024, 6:55 AM
Unknown Object (File)
Aug 26 2024, 10:53 AM
Subscribers

Details

Summary

WNS and Web notifications have almost identical structure (Windows notification doesn't have id field). Therefore we can use the same encryption method for
both. This differentila refactors crypto.js code so that this is possible.

Test Plan
  1. Flow
  2. Test that encrypted Web notitfications work correctly. Windows will be tested in next differential, but since web and windows encryption share encryption the fact

that web encryption works suggests that windows should as well.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin requested review of this revision.Dec 5 2023, 5:13 AM
This revision is now accepted and ready to land.Dec 6 2023, 3:58 AM