add a health endpoint #448
Conversation
Signed-off-by: Andreas Bräu <[email protected]>
|
@digitaldan can you please have a look on that? |
|
Hi, LGTM, i'm working in the code this week, so will give this a quick test today or tomorrow. Thanks! |
|
HI one comment, can you add a config option to enable/disable this? I don't want to have this exposed on our public service, and would prefer not to try and block this specific route in our load balancer (its already complicated). Thanks ! |
Signed-off-by: Andreas Bräu <[email protected]>
|
Sure, I added a config option. Usually things like that I would only expose to another endpoint, that is not public. |
|
@digitaldan can you please take a look again? |
| @@ -6,7 +6,10 @@ | |||
| "logger" : { | |||
| "type": "console" | |||
| }, | |||
| "subDomainCookies": false | |||
| "subDomainCookies": false, | |||
| "healthEndpoint" : { | |||
There was a problem hiding this comment.
Any reason why this can't be a single value, like "healthEndpointEnabled" : true ? Are you thinking there would be more options available in the future for the health endpoint that would warrent its own section/object ?
There was a problem hiding this comment.
yes, for example we could add another option to configure not to return error details, but just the health status.
Or configure, as I said in the other answer to check for other modules/dependencies.
|
Hi, sorry i missed your changes in my sea of github notifications. Just left a few questions. |
|
LGTM, thanks. |
I use openhab cloud in a kubernetes environment. When the mongodb connection gets lost, it doesn't recover alone.
Kubernetes doesn't see that the state isn't healthy by just probing the home page. So I created a health endpoint that returns HTTP status 200 in case every is ok, else it returns HTTP status 500.