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

Add health check to check a URL #491

Closed
sleberknight opened this issue Jun 13, 2024 · 0 comments · Fixed by #505
Closed

Add health check to check a URL #491

sleberknight opened this issue Jun 13, 2024 · 0 comments · Fixed by #505
Assignees
Labels
new feature A new feature such as a new class, method, package, group of classes, etc.
Milestone

Comments

@sleberknight
Copy link
Member

This health check will:

  • check that a URL returns a 2xx response code
  • add a "severity" detail based on the result (uses metrics-healthchecks-severity)
  • allow you to provide your own Client, for example to set up custom TLS settings such as key and trust store
  • provide an "execution condition" that will conditionally execute the health check, for example only when a service instance is the "leader"

Dropwizard has an HttpHealthCheck but it doesn't let you customize anything, and we specifically need the ability to specify custom key and trust stores for the Client. We also need the conditional execution ability.

@sleberknight sleberknight added the new feature A new feature such as a new class, method, package, group of classes, etc. label Jun 13, 2024
@sleberknight sleberknight added this to the 3.7.0 milestone Jun 18, 2024
sleberknight added a commit that referenced this issue Jun 24, 2024
* Add health check to check a URL
* Add a caching health check

Closes #491
Closes #492
@sleberknight sleberknight self-assigned this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature A new feature such as a new class, method, package, group of classes, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant