-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Make access_log for main configurable #5625
Comments
Hi @hafe thanks for reporting! Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂 Cheers! |
Just to mention, I have started working on a pull request. |
Thank you @hafe we will review this issue in our community call. |
related #4341 |
Hi @hafe , We had a chance to discuss this issue and I want to add some context. The future state is to integrate with OpenTelemetry and streaming logs using it. We are planning on it but we do not have a timeline for this work. Until we get to a more robust solution we are happy to accept your contribution with some conditions: 1 - We need a flag to enable the access log Please let us know what you think about it? |
Let's see:
|
Apologies, I should mention that I was looking in the PR. Keep the flag as it is AccessLogOff. Allow the access_log configuration but preventing logging to files. |
Check latest version, only allows syslog |
Tagging @jjngx |
Closes nginx#5625 Signed-off-by: Hans Feldt <[email protected]>
Is your feature request related to a problem? Please describe.
Currently it is not possible to globally configure log forward of access logs in a simple way. Logging high volume access logs to stdout means that they get forwarded to the log stack in Kubernetes. This can be problematic depending on the stack.
Describe the solution you'd like
Add a directive to the global Config map where the main access_log directive can be overridden
Describe alternatives you've considered
An alternative solution is to use a custom main template which is far from simple.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: