fix #547 (comment)

Author: mtrezza

spec/APNSNative.spec.js

@@ -558,6 +558,19 @@ describe('APNSNative', () => {
       expect(result.response.error).toBe('Something wrong happend');
     });
 
+    it('properly parses status 0 transport failures', async () => {
+      spyOn(log, 'error').and.callFake(() => {});
+      const result = await APNSNative._handlePushFailure({
+        device: 'abcd',
+        status: 0,
+        response: { reason: 'RequestTimeout' }
+      });
+      expect(result.transmitted).toBe(false);
+      expect(result.device.deviceToken).toBe('abcd');
+      expect(result.device.deviceType).toBe('ios');
+      expect(result.response.error).toBe('RequestTimeout');
+    });
+
     it('properly parses errors again', async () => {
       spyOn(log, 'error').and.callFake(() => {});
       const result = await APNSNative._handlePushFailure({ device: 'abcd' });

src/APNSNative.js

@@ -338,7 +338,7 @@ export class APNSNative {
     if (failure.error) {
       log.error(LOG_PREFIX, 'APNS error transmitting to device %s with error %s', failure.device, failure.error);
       return APNSNative._createErrorPromise(failure.device, failure.error);
-    } else if (failure.status && failure.response && failure.response.reason) {
+    } else if (failure.status !== undefined && failure.response?.reason) {
       log.error(LOG_PREFIX, 'APNS error transmitting to device %s with status %s and reason %s', failure.device, failure.status, failure.response.reason);
       return APNSNative._createErrorPromise(failure.device, failure.response.reason);
     } else {