bug: APNs library requires parameters to be strings

closes #797
mozilla-services · Feb 6, 2017 · 5348820 · 5348820
1 parent d537fd8
commit 5348820
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/autopush/router/apns2.py b/autopush/router/apns2.py
@@ -11,8 +11,8 @@
 SANDBOX = 'api.development.push.apple.com'
 SERVER = 'api.push.apple.com'
 
-APNS_PRIORITY_IMMEDIATE = 10
-APNS_PRIORITY_LOW = 5
+APNS_PRIORITY_IMMEDIATE = '10'
+APNS_PRIORITY_LOW = '5'
 APNS_MAX_CONNECTIONS = 20
 
 
@@ -107,7 +107,7 @@ def send(self, router_token, payload, apns_id,
             'apns-topic': topic or self.topic,
         }
         if exp:
-            headers['apns-expiration'] = exp
+            headers['apns-expiration'] = str('exp')
         url = '/3/device/' + router_token
         connection = self._get_connection()
         try:

diff --git a/autopush/tests/test_router.py b/autopush/tests/test_router.py
@@ -202,7 +202,7 @@ def test_route_low_priority_notification(self):
         headers = body['headers']
         eq_(headers, {'apns-expiration': exp,
                       'apns-topic': 'com.example.SomeApp',
-                      'apns-priority': 5,
+                      'apns-priority': '5',
                       'apns-id': 'apnsid'})
 
     @inlineCallbacks