-
-
Notifications
You must be signed in to change notification settings - Fork 108
Open
Labels
Description
New Issue Checklist
- I am not disclosing a vulnerability.
- I am not just asking a question.
- I have searched through existing issues.
- I can reproduce the issue with the latest versions of Parse Server and the Parse Server Push Adapter.
Issue Description
The http/2 implementation in firebase-admin throws on network errors instead of handling them, which lets them propagate to the node process, which eventually crashes the process.
- For
GOAWAYerrors a fix has been implemented with Parse Server crash on FCM response GOAWAY #340. - For other network errors such as
ECONNRESETthere is no easy remediation.
The proper solution for any network error is to fix firebase/firebase-admin-node#2789, for example via an event listener.
Steps to reproduce
This issue occurs on network error events, such as ECONNRESET.
Actual Outcome
Unhandled error crashes app.
Expected Outcome
Error should be handled by the firebase-admin.
Workaround
- Use the older version HTTP/1.1 by setting
fcmEnableLegacyHttpTransport: truein your push options. - Catch the error in a global Node process error handler.
Environment
Client
- Parse Server Push Adapter version:
6.8.0
Server
- Parse Server version:
7.3.0