Skip to content

zerologadapter: Pass the context to logger #77

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 1 commit into
base: master
Choose a base branch
from

Conversation

0xjac
Copy link

@0xjac 0xjac commented Jan 8, 2025

  • Pass the context to the logger to be used by hooks.
  • Update test case with a hook to read value from the context.
  • Bump zerolog to v.1.30.0 to use zerolog.Event.Ctx.

Edit: This is useful to retrieve values from the context in hooks to include in logs, such as a trace ID.

    - Pass the context to the logger to be used by hooks.
    - Update test case with a hook to read value from the context.
    - Bump zerolog to v.1.30.0 to use `zerolog.Event.Ctx`.
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