Page MenuHomePhabricator
Differential D9560

Revert "Revert "Implement stateful and deferrable notification decryption and use it on iOS in NSE""
ClosedPublic
Actions

Authored by ashoat on Oct 22 2023, 9:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 10:13 PM
Unknown Object (File)
Mon, Oct 28, 7:18 PM
Unknown Object (File)
Oct 4 2024, 2:00 AM
Unknown Object (File)
Oct 4 2024, 2:00 AM
Unknown Object (File)
Oct 4 2024, 2:00 AM
Unknown Object (File)
Oct 4 2024, 2:00 AM
Unknown Object (File)
Oct 4 2024, 1:58 AM
Unknown Object (File)
Sep 17 2024, 2:32 PM
View All 73 Files
Subscribers
marcin
will

Details

Reviewers
marcin
tomek
bartek
atul
Commits
rCOMM8a79b69fbd94: Revert "Revert "Implement stateful and deferrable notification decryption and…
Summary

This reverts commit 56c0498aa0a3c415243be9097eedd673a6dc38c1.

Depends on D9559

Test Plan

This is bringing back @marcin's D9401. Changes to fix the resulting bug (ENG-5354) will follow in subsequent diffs.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat created this revision.Oct 22 2023, 9:04 AM
Herald added a subscriber: will. · View Herald TranscriptOct 22 2023, 9:04 AM
ashoat added a child revision: D9561: [native] Avoid copy-constructing CryptoModule.Oct 22 2023, 9:05 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Oct 22 2023, 9:06 AM
Harbormaster failed remote builds in B23415: Diff 32296!
ashoat updated this revision to Diff 32300.Oct 22 2023, 9:08 AM

Rebase

Harbormaster completed remote builds in B23419: Diff 32300.Oct 22 2023, 9:32 AM
ashoat requested review of this revision.Oct 22 2023, 9:32 AM
marcin accepted this revision.Oct 23 2023, 2:20 AM

This is bringing back @marcin's D9401. Changes to fix the resulting bug (ENG-5354) will follow in subsequent diffs.

Perhaps, we could reorder diffs to comply to the convention that diffs should't leave the code in broken state even if they won't be landed independently.

This revision is now accepted and ready to land.Oct 23 2023, 2:20 AM
ashoat added a comment.Oct 23 2023, 5:12 AM

I see your point – I'll try to do that next time. In this case, since I'm landing these together as a stack I'm going to skip the extra work of reordering

Closed by commit rCOMM8a79b69fbd94: Revert "Revert "Implement stateful and deferrable notification decryption and… (authored by ashoat). · Explain WhyOct 23 2023, 5:13 AM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM8a79b69fbd94: Revert "Revert "Implement stateful and deferrable notification decryption and….

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
Notifications/
BackgroundDataStorage/
19 lines
59 lines
ios/
NotificationService/
36 lines

Diff 32313

View Options

native/cpp/CommonCpp/Notifications/BackgroundDataStorage/NotificationsCryptoModule.h

Loading...
View Options

native/cpp/CommonCpp/Notifications/BackgroundDataStorage/NotificationsCryptoModule.cpp

Loading...
View Options

native/ios/NotificationService/NotificationService.mm

Loading...