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

How to disable /healthz access log? #435

Closed
shlima opened this issue Oct 28, 2019 · 1 comment · Fixed by #464
Closed

How to disable /healthz access log? #435

shlima opened this issue Oct 28, 2019 · 1 comment · Fixed by #464
Assignees

Comments

@shlima
Copy link

shlima commented Oct 28, 2019

How can I disable /healthz access log?

I was trying to set:

log:
  access_level: "warn"
log:
  access_level: "error"
log:
  access_log: "/dev/null"

But the following lines are always appear in console:

gorush_1       | [GIN] 2019/10/28 - 08:31:06 | 200 |      62.001µs |       127.0.0.1 | GET      /healthz
gorush_1       | [GIN] 2019/10/28 - 08:31:16 | 200 |      51.607µs |       127.0.0.1 | GET      /healthz
gorush_1       | [GIN] 2019/10/28 - 08:31:26 | 200 |      48.127µs |       127.0.0.1 | GET      /healthz
gorush_1       | [GIN] 2019/10/28 - 08:31:37 | 200 |      47.437µs |       127.0.0.1 | GET      /healthz
gorush_1       | [GIN] 2019/10/28 - 08:31:47 | 200 |      46.996µs |       127.0.0.1 | GET      /healthz
@appleboy
Copy link
Owner

appleboy commented Nov 2, 2019

I will take it.

@appleboy appleboy self-assigned this Nov 2, 2019
appleboy added a commit that referenced this issue Feb 5, 2020
Add zerolog for gin httpd log

fixed: #435

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
appleboy added a commit that referenced this issue Feb 5, 2020
* chore(log): ignore /healthz request log

Add zerolog for gin httpd log

fixed: #435

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

* remove variable

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants