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

Create an informative route /health that can be used in health checks #1699

Closed
bschmalhofer opened this issue Mar 29, 2022 · 3 comments
Closed
Milestone

Comments

@bschmalhofer
Copy link
Contributor

Give some indication that the web server is alive. This could also be used by the Docker Compose health checks.
See also:

See also RotherOSS/otobo-docker#12 .

Elasticsearch does not follow the above guide lines and emits the following:

otobo@fbf22e14524c:~$ curl -v -s -f http://elastic:9200/_cat/health
*   Trying 172.18.0.3:9200...
* Connected to elastic (172.18.0.3) port 9200 (#0)
> GET /_cat/health HTTP/1.1
> Host: elastic:9200
> User-Agent: curl/7.74.0
> Accept: */*
> 
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< X-elastic-product: Elasticsearch
< Warning: 299 Elasticsearch-7.16.2-2b937c44140b6559905130a8650c64dbd0879cfb "Elasticsearch built-in security features are not enabled. Without authentication, your cluster could be accessible to anyone. See https://www.elastic.co/guide/en/elasticsearch/reference/7.16/security-minimal-setup.html to enable security."
< content-type: text/plain; charset=UTF-8
< content-length: 66
< 
1648550638 10:43:58 docker-cluster green 1 1 6 6 0 0 0 0 - 100.0%
* Connection #0 to host elastic left intact
otobo@fbf22e14524c:~$ 


@bschmalhofer bschmalhofer added this to the Wishlist milestone Mar 29, 2022
@bschmalhofer
Copy link
Contributor Author

I'm not sure whether leaking status information is a good thing. So let's just add the route health with dummy content. That rounte can be used in health checks instead of the currently used route robots.txt .

@bschmalhofer
Copy link
Contributor Author

Route has been added, but it is not yet used in healthcheck. Closing this issue.

@bschmalhofer
Copy link
Contributor Author

Moved to the mile stone to OTOBO 11.0.1 because the feature is included there.

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

No branches or pull requests

1 participant