Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable logs for health check endpoint #1896

Closed
d-uzlov opened this issue Oct 18, 2023 · 2 comments · Fixed by #1897
Closed

Disable logs for health check endpoint #1896

d-uzlov opened this issue Oct 18, 2023 · 2 comments · Fixed by #1897
Milestone

Comments

@d-uzlov
Copy link

d-uzlov commented Oct 18, 2023

Summary

Either disable logs for health check endpoint completely or add an environment variable to disable it.

Use case

I want to setup frequent health checks.
However, each health check generates a new log line Access.INFO - GET /rest/health 200 120. It's hard to find anything in logs when they are filled with health check entries.

@acelaya
Copy link
Member

acelaya commented Oct 19, 2023

Yep, this makes sense. It's a common practice.

I think it should be ok to just disable the logs for this endpoint. No need for an env var.

@acelaya
Copy link
Member

acelaya commented Oct 20, 2023

This is now implemented and will be released with v3.7.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants