-
-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GOAWAY errors #140
Comments
Thanks for opening this issue! |
Looks like a rate limit, or maybe the stream is just closed. |
Thanks for looking into this! If I look at the error codes that APNS should be returning I would expect some more information if running into a rate limit. I can't really find anything else about this error apart from some fix in an unmaintained version of this project. Do you know any other way I can debug why this is happening and perhaps prevent this in the future? |
If anyone wants to open a PR here with the fixes, then we can merge. |
I believe those fixes are already implemented in this library. |
Got it, in that case it would be great if someone who is experiencing this issue could investigate some more, possibly manually add a log line to where this issue occurs so we can get more info in case the apn library swallows some of the error information. |
We have consistently been getting this error on about 1 out of 100 notifications we sent. Any idea how we can fix this?
The text was updated successfully, but these errors were encountered: