Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Check for response status #22

Closed
AnandChowdhary opened this issue Aug 28, 2020 · 1 comment · Fixed by #23
Closed

Check for response status #22

AnandChowdhary opened this issue Aug 28, 2020 · 1 comment · Fixed by #23

Comments

@AnandChowdhary
Copy link
Contributor

It'd be great if we could fail the status check if the server responds with a 40x or 50x status rather than an expected 20x repsonse.

Screen Shot 2020-08-28 at 16 48 20

In this example, I had no way of knowing that this webhook was triggered but the server threw an error.

@joelwmale
Copy link
Owner

Your branch has been merged and released. Thanks again for your help on the project @AnandChowdhary!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants