Skip to content

Cleanup nginx logs #7888

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

benglewis
Copy link
Contributor

@benglewis benglewis commented Jul 1, 2025

  • The NGINX_ERROR_LOG_LEVEL environment variable was introduced in deploy/docker-entrypoint.d/common/10-configure-nginx.sh to control the error_log level in deploy/default.conf (i.e. the nginx configuration).
  • A NGINX_LOGGING.md file was introduced to document this environment variable.

A little context: We are seeing lots of noisy logs that look like this today:

2025/07/01 11:20:27 [info] 20#20: *692917 client 35.xyz.xyz.xyz closed keepalive connection

This new environment variable will allow us to silence those logs

@benglewis benglewis requested a review from a team as a code owner July 1, 2025 12:54
Copy link

netlify bot commented Jul 1, 2025

👷 Deploy request for heartex-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 582de65

Copy link

netlify bot commented Jul 1, 2025

👷 Deploy request for label-studio-docs-new-theme pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 582de65

Copy link

netlify bot commented Jul 1, 2025

Deploy Preview for label-studio-playground failed. Why did it fail? →

Name Link
🔨 Latest commit 582de65
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/6863da7efc9038000864440e

Copy link

netlify bot commented Jul 1, 2025

Deploy Preview for label-studio-storybook failed. Why did it fail? →

Name Link
🔨 Latest commit 582de65
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/6863da7eda1eaf00080c2184

@farioas
Copy link
Member

farioas commented Jul 3, 2025

Could you please try to experiment with proxy_ignore_client_abort on; setting for your case?
If it works let's use it without any conditionals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants