Page MenuHomePhabricator
Differential D9068

Send blob holder in large notification payload
AbandonedPublic
Actions

Authored by marcin on Sep 1 2023, 9:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 8 2024, 5:11 AM
Unknown Object (File)
Mar 18 2024, 5:57 PM
Unknown Object (File)
Mar 7 2024, 2:38 AM
Unknown Object (File)
Mar 6 2024, 11:47 PM
Unknown Object (File)
Mar 6 2024, 11:36 PM
Unknown Object (File)
Mar 6 2024, 10:25 PM
Unknown Object (File)
Dec 15 2023, 2:18 AM
Unknown Object (File)
Dec 15 2023, 2:14 AM
View All 18 Files
Subscribers
ashoat

Details

Reviewers
tomek
bartek
atul
Summary

This differential adds blob holder to notification payload which full content exceeds APN/FCM limits. This way native clients will have data necessary to issue blob deletion once they have downloaded the blob.

Test Plan

Run flow.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin created this revision.Sep 1 2023, 9:25 AM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 1 2023, 9:25 AM
marcin added a parent revision: D8698: Upload iOS notification payload to blob service if its size exceeds limits.Sep 1 2023, 9:26 AM
marcin added a child revision: D9069: Schedule blob deletion on Android when large notifications arrives.Sep 1 2023, 9:36 AM
marcin edited parent revisions, added: D8589: Upload Android notification payload to blob service if its size exceeds limits; removed: D8698: Upload iOS notification payload to blob service if its size exceeds limits.
Harbormaster completed remote builds in B22352: Diff 30706.Sep 1 2023, 9:43 AM
marcin requested review of this revision.Sep 1 2023, 9:43 AM
tomek accepted this revision.Sep 5 2023, 4:09 AM
tomek added inline comments.
keyserver/src/push/send.js
812–817 ↗(On Diff #30706)

A shorthand can be used

This revision is now accepted and ready to land.Sep 5 2023, 4:09 AM
marcin updated this revision to Diff 30794.Sep 6 2023, 4:12 AM

Rebase

Harbormaster completed remote builds in B22402: Diff 30794.Sep 6 2023, 4:42 AM
marcin abandoned this revision.Mar 19 2024, 8:29 AM

Those changes will be included in updated version of child differential.

marcin removed a child revision: D9069: Schedule blob deletion on Android when large notifications arrives.Mar 19 2024, 8:30 AM

Revision Contents
Changeset List

PathSize
keyserver/
src/
push/
12 lines
1 line
6 lines

Diff 30794

View Options

keyserver/src/push/send.js

Loading...
View Options

keyserver/src/push/types.js

Loading...
View Options

keyserver/src/push/utils.js

Loading...