Conversation
jschr
force-pushed
the
disable-logging-by-default
branch
from
bbc7358 to
c99152e
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
c99152e to
000ba14
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
3288f82 to
596a21f
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
596a21f to
8cd9715
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
8cd9715 to
2e45fa5
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
2e45fa5 to
3381623
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
3381623 to
ba479fd
Compare
jschr
force-pushed
the
disable-logging-by-default
branch
from
ba479fd to
3dab462
Compare
jschr
commented
Nov 10, 2025
| proxy_cache_valid 200 206 1s; | ||
|
|
||
| access_log ${LOG_DIR}/cache.log proxy_cache; | ||
| ${CACHE_ACCESS_LOG_DIRECTIVE} |
Member
Author
There was a problem hiding this comment.
Will be access_log ${LOG_DIR}/cache.log proxy_cache; when ENABLE_ACCESS_LOGS=true.
Will be access_log off; when ENABLE_ACCESS_LOGS is not set (default).
jschr
commented
Nov 10, 2025
| proxy_ssl_server_name on; | ||
|
|
||
| access_log ${LOG_DIR}/range_cache.log range_cache; | ||
| ${RANGE_ACCESS_LOG_DIRECTIVE} |
Member
Author
There was a problem hiding this comment.
Will be access_log ${LOG_DIR}/range_cache.log range_cache; when ENABLE_ACCESS_LOGS=true.
Will be access_log off; when ENABLE_ACCESS_LOGS is not set (default).
jschr
commented
Nov 10, 2025
|
|
||
| access_log ${LOG_DIR}/access.log; | ||
| error_log ${LOG_DIR}/error.log; | ||
| ${ACCESS_LOG_DIRECTIVE} |
Member
Author
There was a problem hiding this comment.
Will be access_log ${LOG_DIR}/access.log; when ENABLE_ACCESS_LOGS=true.
Will be access_log off; when ENABLE_ACCESS_LOGS is not set (default).
jschr
commented
Nov 10, 2025
| access_log ${LOG_DIR}/access.log; | ||
| error_log ${LOG_DIR}/error.log; | ||
| ${ACCESS_LOG_DIRECTIVE} | ||
| error_log stderr ${ERROR_LOG_LEVEL}; |
Member
Author
There was a problem hiding this comment.
Instead log errors to stderr to be more compabitble with running in Docker.
ERROR_LOG_LEVEL is crit by default.
Jabba
approved these changes
Nov 12, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Disable access logs by and send critical errors to stdout to prevent noisy web apps from causing unbounded log storage before the daily log rotate.
Access logs can be enabled again by setting
ENABLE_ACCESS_LOGS=trueand the error log level can be modified withERROR_LOG_LEVEL.