Add OpenTelemetry along side with OpenTracing

Byndyusoft / node-pino-logger-factory

:evergreen_tree: logger factory for pino :evergreen_tree:

Apache License 2.0

0 stars 0 forks source link

Add OpenTelemetry along side with OpenTracing #136

Open KillWolfVlad opened 1 year ago

KillWolfVlad commented 1 year ago

For traceId

nmorgunov commented 1 year ago

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

KillWolfVlad commented 1 year ago

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