Skip to content
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

feat(status): custom status codes #190

Merged
merged 3 commits into from Jun 4, 2021
Merged

feat(status): custom status codes #190

merged 3 commits into from Jun 4, 2021

Conversation

gergelyke
Copy link
Collaborator

No description provided.

@gergelyke
Copy link
Collaborator Author

@sindilevich would you mind taking it from here and create a PR that passes CI? sadly, I cannot find the time to finish this up

@sindilevich sindilevich mentioned this pull request Jun 2, 2021
@sindilevich
Copy link
Contributor

@gergelyke, tried to overcome the failing tests with #191

@sindilevich
Copy link
Contributor

@rxmarbles, any chance you can take a look at #191, which fixes this PR, and adds tests and typings, please?

* Revert "feat(status): custom status codes"

This reverts commit 522df64.

* Added statusOk and statusError to configuration

The new configuration properties: statusOk and statusError will allow to globally configure the HTTP codes, used to respond with on a successful or an unsuccessful healthcheck respectively.
An unsuccessful healthcheck may override the global statusError property by setting the statusCode property on the Error object.

* Added typings for statusOk and statusError
@rxmarbles rxmarbles merged commit ee94766 into main Jun 4, 2021
@rxmarbles rxmarbles deleted the custom-status branch June 4, 2021 14:29
@github-actions
Copy link

github-actions bot commented Jun 4, 2021

🎉 This PR is included in version 4.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

3 participants