Byndyusoft / node-pino-logger-factory

:evergreen_tree: logger factory for pino :evergreen_tree:

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add OpenTelemetry along side with OpenTracing

KillWolfVlad opened this issue · comments

For traceId

Я тут подумал. Почему кубик логгера должен знать что-то о трассировке? Корректна ли эта связность? Может это стоит оставить на пользователя? Ручка для этого есть: withCustomProps

Надо посмотреть как это будет в OpenTelemetry, и исходя из этого уже принимать финальное решение. Хочется чтобы кубик для настройки логгера предоставлял нам крутые дефолты (в том числе traceId, если он есть), для этого он и был создан.