Repository navigation
fix(nginx): LOGLEVEL=notice — opóźnienia bpp_ip_slow trafiają do logu - #39
Merged
Merged
Conversation
Obraz owasp/modsecurity-crs:nginx ma error_log na ${LOGLEVEL}=warn, a nginx
loguje OPOZNIENIA limit_req o poziom nizej niz odmowy (notice przy
limit_req_log_level warn). Linie "delaying request ... bpp_ip_slow" ginely:
make rate-limit-stats pokazywal "spowolnieni: 0" mimo spowalniania, a
make test-nginx-limits padal w CI (tez na main) po pullu nowszego obrazu.
- webserver: LOGLEVEL=notice (notice -> Alloy mapuje na info, nie error),
- test-nginx-limits: ten sam LOGLEVEL + LC_ALL=C przy porownaniu czasow
(pl_PL z przecinkiem dziesietnym liczyl opoznienia zle),
- test_nginx_global_limits_wired: asercja LOGLEVEL w compose (sprawdzona
mutacja warn -> FAIL), bo test runtime ustawia go sam.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Problem
Zadanie CI Testy runtime padało (także na
main):make test-nginx-limits→brak linii 'delaying request ... bpp_ip_slow' w logu.Przyczyna nie leży w teście. Obraz
owasp/modsecurity-crs:nginx(tag pływający, build 2026-10-18) maerror_log ${ERRORLOG} ${LOGLEVEL}zLOGLEVEL=warn, a nginx loguje opóźnienialimit_reqo poziom niżej niż odmowy, czyli przylimit_req_log_level warnnanotice. Na produkcjimake rate-limit-statspokazywał przez to „spowolnieni: 0”, mimo że nginx spowalniał klientów.Zmiany
webserver:LOGLEVEL: "${LOGLEVEL:-notice}". W Alloynotice→info, więc linie nie trafiają na dashboard błędów. Odrzuciłemlimit_req_log_level error, bo odmowy 429 wpadłyby wtedy do błędów.scripts/test-nginx-limits.sh: ten samLOGLEVELorazLC_ALL=Cprzy porównaniu czasów (przypl_PLawk źle liczył opóźnienia).test_nginx_global_limits_wired: statyczna asercjaLOGLEVELw compose, bo test runtime ustawia go sam. Sprawdzona mutacjąwarn→ FAIL.docs/architektura/rate-limiting.md, CLAUDE.md.Testy
make test-nginx-limits(lokalnie, świeży obraz): przed zmianą 2 niezgodności, po zmianie „wszystko zgodne”.tests/test_makefile.sh </dev/null: 404 passed, 0 failed, 1 skipped.mkdocs build --strict: OK.Wdrożenie: zmiana w
environment:, więc wymagamake up(samgit pullnie wystarczy).🤖 Generated with Claude Code