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

Custom status #191

Merged
merged 3 commits into from Jun 4, 2021
Merged

Custom status #191

merged 3 commits into from Jun 4, 2021

Conversation

sindilevich
Copy link
Contributor

@sindilevich sindilevich commented Jun 2, 2021

Solves #176

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.
@rxmarbles rxmarbles merged commit 47608c6 into godaddy:custom-status Jun 4, 2021
rxmarbles added a commit that referenced this pull request Jun 4, 2021
* feat(status): custom status codes

* revert(binary) revert lockfile change

* Custom status (#191)

* 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

Co-authored-by: rxmarbles <rmarkins@gmail.com>
Co-authored-by: Matanel Sindilevich <sindilevich@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants