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

feat: support extended debug levels #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hishamhm
Copy link
Contributor

@hishamhm hishamhm commented Apr 9, 2024

Expose seven additional Nginx log levels that are not normally exposed by OpenResty, which allows for finer-grained debugging with less noise in the debug-level logs: debug_core, debug_alloc, debug_mutex, debug_event, debug_http, debug_mail, debug_stream.

By setting a quieter value such as debug_mail, Kong Gateway debug logs are usable again, as the Kong (and WasmX) logs are still shown, but the other Nginx debug logs (memory allocations, etc.) are not shown.

This PR is a requirement for the correspondent feature in Kong Gateway: Kong/kong#12842

Expose [seven additional Nginx log levels](https://github.com/nginx/nginx/blob/release-1.25.4/src/core/ngx_log.h#L26-L32)
that are not normally exposed by OpenResty, which allows for finer-grained
debugging with less noise in the debug-level logs: `debug_core`,
`debug_alloc`, `debug_mutex`, `debug_event`, `debug_http`, `debug_mail`,
`debug_stream`.

By setting a quieter value such as `debug_mail`, Kong Gateway debug logs are
usable again, as the Kong (and WasmX) logs are still shown, but the other
Nginx debug logs (memory allocations, etc.) are not shown.
hishamhm added a commit to Kong/kong that referenced this pull request Apr 9, 2024
Dynamically enabling the extended debug log levels requires a
new version of lua-kong-nginx-module which incorporates
Kong/lua-kong-nginx-module#87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant