cashapp / misk

Microservice Kontainer
https://cashapp.github.io/misk/
Apache License 2.0
407 stars 170 forks source link

Add deprecated annotation to TaggedLogger #3365

Closed afkelsall closed 3 months ago

afkelsall commented 4 months ago

We are refactoring the TaggedLogger within misk to be less opinionated and complicated to use. Under the new implementation, services will still be able to access the original functionality of this logger to add tags to logs and include them on underlying web exceptions thrown by misk.

This PR is just to mark this implementation as deprecated to hint that it shouldn't be used yet.

adrw commented 3 months ago

Merged in https://github.com/cashapp/misk/commit/d13f8e10fa64c45c5a0a9574730ec02bb642e85f