qbit / node-pushover

Pushover notifications for node.js (JavaScript,NodeJS)
ISC License
163 stars 24 forks source link

Don't crash when JSON parsing fails #26

Closed sulkaharo closed 5 years ago

sulkaharo commented 5 years ago

Pushover has been under DDOS today and responds with a HTML page with a 5xx status code, which doesn't parse as JSON, resulting in this library crashing. This PR prevents that crash

qbit commented 5 years ago

Sweet! ty!

sulkaharo commented 5 years ago

Great! Can you publish the updated package with a minor version change, so new deploys get the fix automatically?

qbit commented 5 years ago

Just pushed 1.2.1