Skip to content

Parse Sever Crash When .p12 files missing #2117

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

Closed
acegreen opened this issue Jun 21, 2016 · 5 comments
Closed

Parse Sever Crash When .p12 files missing #2117

acegreen opened this issue Jun 21, 2016 · 5 comments
Labels
type:feature New feature or improvement of existing feature

Comments

@acegreen
Copy link

acegreen commented Jun 21, 2016

Issue Description

I just finished updating my server and didn't have the push notification .p12 files on the new instance. When I tried to send a push notification using the server, not only did it not go through, but the server instance crashed.

Steps to reproduce

  1. Add the certificate properties in your index.js file
  2. Do not have your .p12 files in the location specified
  3. Send a push

Expected Results

Verbose about missing .p12 files and no pushes actually sent out ONLY

Actual Outcome

Verbose and server crash

Environment Setup

  • Server
    • parse-server version: 2.2.13
    • Operating System: Ubuntu 15.10
    • Hardware: 4 GB Memory / 60 GB Disk / NYC3
    • Localhost or remote server? (AWS, Heroku, Azure, Digital Ocean, etc): Digital Ocean
  • Database
    • MongoDB version: 3.2.7
    • Storage engine: N/A
    • Hardware: 4 GB Memory / 60 GB Disk / NYC3
    • Localhost or remote server? (AWS, mLab, ObjectRocket, Digital Ocean, etc): Digital Ocean

Logs/Trace

Didn't keep it

@hramos
Copy link
Contributor

hramos commented Jun 22, 2016

Was there no output when the server crashed? Wondering how obscure the issue would be when someone runs into this, and if its worth handling as a special case.

@acegreen
Copy link
Author

There was but I was in a rush to fix it because my production app relies on the server. I think this is not obscure at all, just remove the .p12 files from the location specified in your index.js and send a push.

@jeffreyjackson
Copy link

I ran into an issue today where the push certificate had expired and as a result parse server was unresponsive and continued to crash.

@flovilmart
Copy link
Contributor

Closing as it’s an issue in the node-APN module. If you want it fixed, open the issue there

@jeffreyjackson
Copy link

I found the issue on the parse server push adapter module. Thanks!

@mtrezza mtrezza added type:feature New feature or improvement of existing feature and removed type:improvement labels Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New feature or improvement of existing feature
Projects
None yet
Development

No branches or pull requests

5 participants