search

zio / zio-logging

Powerful logging for ZIO 2.0 applications, with compatibility with many logging backends out-of-the-box.
https://zio.dev/zio-logging/
Apache License 2.0
174 stars 83 forks source link

Add appendLoggerName aspect #893

Closed somdoron closed 4 weeks ago

somdoron commented 4 weeks ago

We are upgrading from ZIO 1, where the previous version of zio.logging appended the logger name to the existing logger name. In the new version, it replaces it. This PR suggests introducing a new aspect, appendLoggerName, to append a string to the existing logger name.

Our codebase relies on setting the logger name. For example, when our RPC framework receives an incoming message, we add the service name and method name to the logger name.

justcoon commented 4 weeks ago

@somdoron thanks for contribution, i have to take a look on CI issues first ..., as builds are failing currently

justcoon commented 4 weeks ago

hi @somdoron, i think you need to run scalafix