Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

AM-326 ams:Introduce trace id identifier in logs #458

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

agelostsal
Copy link
Contributor

Added a new parameter that should be present in all logs that take place after a user has executed any request against the system.
The trace_id parameter will assist in grouping logs of the same originator making it easier to check what happened during a specific flow.

The most idiomatic way to add request-scoped parameters is through the standard context package.
It's common in go to propagate context as a first parameter in each function.

The biggest change is the store interface which had all its methods to accept a context as first argument.
This change also helps in the migration to the new mongo official driver, which expects context on all its calls.

@agelostsal agelostsal force-pushed the logs-ench/trace-id branch from 148e77d to 1f640cb Compare June 27, 2023 09:05
@themiszamani themiszamani merged commit 597f3a4 into ARGOeu:devel Sep 26, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants