Enhance logs construction and usage in aggregator #2002
Merged
+917
−518
Commits on Oct 11, 2024
-
refactor(aggregator): simplify
mockall::automockimportsUse full path instead of needing a use gated by an `cfg(test)` that add noises with the other imports.
-
feat(common): new logger extension to create logger with given name
To create child logger when there's no component to hold them (such as the http server in aggregator).
-
refactor(aggregator): add a child logger to all components that use logs
* Tagged with the component name. * Existing logs are still using `slog_scope` but this prepare its removal.
-
refactor(aggregator): use dedicated logger instead of
slog_scopein…… http server To tag the logs with a source so they may be easily filtered.
-
refactor(aggregator): use dedicated logger instead of
slog_scopein…… services & tools To tag the logs with a source so they may be easily filtered.
Commits on Oct 14, 2024
-
test(aggregator): use simpler method to build sqlite connection in `c…
…ertificate_repository`
-
-
chore: remove
slog_scopefrom aggregator main dependenciesIt's still a `dev-dependencies` as it is useful for integration tests.
-
style(aggregator): set application name in log to
mithril-aggregatorInstead of the default `slog-rs`.
-
* mithril-aggregator from `0.5.80` to `0.5.81` * mithril-common from `0.4.67` to `0.4.68`
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.