Paths

Table of Contentst

[Tunnelbroker][lib] add APNs notif message type
Needs ReviewPublic
Actions

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

Details

Reviewers

bartek
marcin

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)

Revision Contents
Changeset List

Path

Size

lib/

types/

tunnelbroker/

messages.js

5 lines

notif-types.js

9 lines

shared/

tunnelbroker_messages/

src/

messages/

mod.rs

3 lines

notif.rs

14 lines

Diff 41793

View Options

lib/types/tunnelbroker/messages.js

View Options

lib/types/tunnelbroker/notif-types.js

View Options

shared/tunnelbroker_messages/src/messages/mod.rs

View Options

shared/tunnelbroker_messages/src/messages/notif.rs

[Tunnelbroker][lib] add APNs notif message typeNeeds ReviewPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 41793

lib/types/tunnelbroker/messages.js

lib/types/tunnelbroker/notif-types.js

shared/tunnelbroker_messages/src/messages/mod.rs

shared/tunnelbroker_messages/src/messages/notif.rs

[Tunnelbroker][lib] add APNs notif message type
Needs ReviewPublic
Actions

Revision Contents
Changeset List