Skip to content

Canonical log#30

Open
nofun97 wants to merge 6 commits intoanz-bank:masterfrom
nofun97:canonical-log
Open

Canonical log#30
nofun97 wants to merge 6 commits intoanz-bank:masterfrom
nofun97:canonical-log

Conversation

@nofun97
Copy link
Contributor

@nofun97 nofun97 commented Feb 18, 2020

Added API for canonical log and tests.

This is part of #16

@nofun97 nofun97 requested a review from anzdaddy February 18, 2020 02:18
@daemonl
Copy link

daemonl commented Feb 19, 2020

I think this problem could be solved in a different and possibly simpler way by implementing it as a package on top of the log library.

Imagine a library which exposes:

  • A method to add data to the context
  • A middleware to set up the context before passing down, then read the values from the context and actually log at the end of the request

It would be able to be achieved with no modification to the logging library itself, and then becomes optional to follow the exact pattern of one log line per client request depending on the needs of the consumer

anzopensource pushed a commit that referenced this pull request Feb 7, 2023
* Upgrade otel metrics to v0.35

* Tidy!
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.

3 participants