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

net/ghttp: conditional printing access log to stdout #3562

Closed
wants to merge 1 commit into from

Conversation

chamhaw
Copy link

@chamhaw chamhaw commented May 6, 2024

Added AccessLogStdoutDisabled toggle to disable access content logging into stdout and supported conditional access log to exclude endpoints such as /health, /metrics etc.

feat:add AccessLogStdoutDisabled toggle to disable access content logging into stdout and support conditional access log
@gqcn
Copy link
Member

gqcn commented May 7, 2024

@chamhaw Hi, the logging content to stdout can be disabled though configuration LogStdout. The logging content should not be filtered by configuration, that is, the logging content should be complete or nothing but not be filtered manually.

@gqcn gqcn added the rejected The proposal or PR is not accepted, which might be conflicted with our design or plan. label May 7, 2024
@gqcn gqcn closed this May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejected The proposal or PR is not accepted, which might be conflicted with our design or plan.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants