Page MenuHomePhabricator

[Tunnelbroker] implement creating APNs client
AcceptedPublic

Authored by kamil on Fri, Jun 28, 5:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 29, 12:47 AM
Unknown Object (File)
Sat, Jun 29, 12:47 AM
Unknown Object (File)
Sat, Jun 29, 12:46 AM
Subscribers

Details

Reviewers
bartek
marcin
Summary

Implementing APNs client which will handle sending notifs.

After some research, I discovered that opening new connections for every request could be treated as a DoS attack. However, reqwest can handle that, if we use the same Client and when making a request it will use keep-alive connection pooling.

Depends on D12610

Test Plan

Test creating client with adding config args from command-line args.

Diff Detail

Repository
rCOMM Comm
Branch
apns-tb
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil retitled this revision from [Tunnelbroker] implement creating APNs token to [Tunnelbroker] implement creating APNs client.Fri, Jun 28, 5:01 AM
kamil edited the summary of this revision. (Show Details)
kamil published this revision for review.Fri, Jun 28, 6:27 AM
kamil edited the summary of this revision. (Show Details)
kamil edited the test plan for this revision. (Show Details)
bartek added inline comments.
services/tunnelbroker/src/notifs/apns/mod.rs
20–24

Are these requirements described anywhere in Apple docs?

This revision is now accepted and ready to land.Mon, Jul 1, 2:13 AM