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.

Sign up for GitHub

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

Jump to bottom

feat(request-id): add a request ID to logs and error templates #11308

Closed
wants to merge 5 commits into from
Closed

feat(request-id): add a request ID to logs and error templates #11308

wants to merge 5 commits into from

Commits on Sep 19, 2023

  1. feat(request-id): add Request ID 

    * Add a request ID (which matches the trace_id or correlation_id when
  available) to error templates and logs.

* Bump lua-kong-nginx-module to version 0.7.0

* Use the new directive `lua_kong_error_log_request_id`
  introduced in 0.7.0 which adds the request id to the error log output
    @samugi
    samugi committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    2bf710e View commit details
    Browse the repository at this point in the history

  2. feat(request-id): unit tests and PR feedback 

    add unit tests
add integration tests to cover zipkin as well
make access_log include request_id by default
refactoring and log messages
    @samugi
    samugi committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    cff40dd View commit details
    Browse the repository at this point in the history

  3. feat(request_id): default error templates 

    add request id to the default error templates
    @samugi
    samugi committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    24649b3 View commit details
    Browse the repository at this point in the history

  4. feat(request-id): add request id to log serializer

    @samugi
    samugi committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    267ec14 View commit details
    Browse the repository at this point in the history

  5. feat(request-id): move trace id set in propagation + bump 

    setting trace id from propagation allows to set the correct format
of trace id depending on the headers format, instead of defauting to hex

bump lua-kong-nginx-module to 0.7.1
    @samugi
    samugi committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    d280b41 View commit details
    Browse the repository at this point in the history