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

tracing-actix-web-mozlog: Semantic concerns (like adding request ID to the fields) should not be mixed with formatting concerns (logging as JSON) #27

Open
mythmon opened this issue Nov 3, 2021 · 0 comments

Comments

@mythmon
Copy link
Contributor

mythmon commented Nov 3, 2021

Currently the JSON formatter performs several transformations of the data beyond formatting it as compatible JSON. This can be a problem for debugging or other environments that want to use human readable logs, but test the transformations that make this valid mozlog.

We should separate the two sets of concerns into two or more independent layers that can each be used separately.

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

No branches or pull requests

1 participant