-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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
UseHealthChecks vs. MapHealthChecks /5 #29439
Conversation
@JuergenGutsch please review. It's way too long, should I cut most of that out? |
@ngbrown please review |
@CarlosLanderas @jods4 @mettelephant @FirstAfterGod2501 @bhugot please review |
@Rick-Anderson I think the updated explanation looks good. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@JuergenGutsch please review. It's way too long, should I cut most of that out? |
I seems like in .NET 8 we will be able to write |
This comment was marked as resolved.
This comment was marked as resolved.
* Provides a default health check without specifying a path. Allows performing a health check on the root path or any request made to the specified port. | ||
* Selecting any endpoint on a port. | ||
* Provides a catch-all health check. Handles health checks for any path. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are all saying the same thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are all saying the same thing.
@JamesNK Can you make a suggestion of the lines to delete?
Co-authored-by: James Newton-King <james@newtonking.com>
Co-authored-by: James Newton-King <james@newtonking.com>
Fixes #29421
Internal previews