Automattic / pushpress

https://wordpress.org/plugins/pushpress
14 stars 9 forks source link

HTTP 200 OK is also a success #8

Closed da2x closed 2 years ago

dd32 commented 2 years ago

I'm closing this as it seems to be incorrect.

The branch of code in question is specifically only for not-2xx requests, changing it from < to <= would make it fire for anything other than 201-299 which doesn't seem correct.

This change would result anything that responds with 200 being unsubscribed from future pings.