ServiceStatus success about vapor-apns HOT 2 CLOSED

Possibly related: I think there's something seriously wrong with reporting the result. Few days back, I was debugging push notifications suddenly not getting delivered even though I was getting the .success result. Turns out they were never accepted by APNS servers due to a typo that occurred in the private or public key, can't remember which one. At any rate, neither this case or the one originally reported definitely shouldn't end up with a .success result case.

For the record, I'm using the new HTTP/2 method of connecting to APNS (just with private and public key manually extracted from the p8 file, to be able to supply it as two environmental variables), if that changes anything.

from vapor-apns.

I just found the cause of the bug, will try to fix it when I have time.

if responseData != "" {
    // Get JSON from loaded data string
    let jsonNode = JSON(.bytes(responseData.makeBytes()), in: nil).makeNode(in: nil)
    if let reason = jsonNode["reason"]?.string {
        result = Result.error(apnsId: message.messageId, deviceToken: deviceToken, error: APNSError.init(errorReason: reason))
    } else {
        result = Result.success(apnsId: message.messageId, deviceToken: deviceToken, serviceStatus: .success)
    }
} else {
    result = Result.success(apnsId: message.messageId, deviceToken: deviceToken, serviceStatus: .success)
}

Here, the response can't be parsed and jsonNode["reason"] is nil, so the result is always success.

Edit: Just realized that #50 fixes the problem.

from vapor-apns.