[Tunnelbroker][lib] add APNs notif message type
ClosedPublic
Actions

Authored by kamil on Fri, Jun 28, 5:41 AM.

Details

Reviewers

bartek
marcin

Commits

rCOMM7c0f632a8b8b: [Tunnelbroker][lib] add APNs notif message type

Summary

Message send from client to send notif using APNs.

As a response to that message using MessageToDeviceRequestStatus which is exactly what we need - follow-up task to rename this to something more generic (ENG-8609).

Depends on D12614

Test Plan

Tested in D12617.

Diff Detail

Repository

rCOMM Comm

Branch

apns-tb

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Fri, Jun 28, 5:41 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFri, Jun 28, 5:41 AM

kamil added a child revision: D12616: [Tunnelbroker] implement sending APNs notifs by client.Fri, Jun 28, 5:46 AM

kamil edited the test plan for this revision. (Show Details)Fri, Jun 28, 5:52 AM

Harbormaster completed remote builds in B29994: Diff 41793.Fri, Jun 28, 5:57 AM

kamil published this revision for review.Fri, Jun 28, 6:33 AM

kamil edited the summary of this revision. (Show Details)

Are any validator changes necessary here?

marcin accepted this revision.Tue, Jul 2, 2:10 AM

This revision is now accepted and ready to land.Tue, Jul 2, 2:10 AM

In D12615#357420, @ashoat wrote:

Are any validator changes necessary here?

No, this is a message from the device to Tunnelbroker only so validation is handled on the Rust side by serde. This could be confusing, some additional context in point 5 in ENG-8621.

Closed by commit rCOMM7c0f632a8b8b: [Tunnelbroker][lib] add APNs notif message type (authored by kamil). · Explain WhyTue, Jul 2, 8:35 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM7c0f632a8b8b: [Tunnelbroker][lib] add APNs notif message type.