-
Notifications
You must be signed in to change notification settings - Fork 8
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
docs(docs): Add ADR on structured logging. #1018
Conversation
✅ Test Report | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds like a good idea. Are we going to have concrete specs for this later?
For example, we said, Date and Time *MUST* always be referenced to UTC and not Local Time.
But what exact format are we going to use? For example, we'd prefer ISO 8601,
not RFC 1123
.
docs/src/architecture/09_architecture_decisions/0008-structured-logging.md
Outdated
Show resolved
Hide resolved
Yes we can. The only reason I didn't is because I wasn't sure what would be easy across all the languages. For Rust we are using rfc_3339 (a subset of ISO 8601). |
…atalyst-voices into adr/structured_logging
docs/src/architecture/09_architecture_decisions/0008-structured-logging.md
Outdated
Show resolved
Hide resolved
…in endpoint deprecation, add note about security requirements.
…st-voices into adr/api_endpoints
Description
Adds an ADR about structured logging.
Please confirm the following checks