aigoncharov 4 мар 2019 в 08:44

Как логировать в NodeJS, чтобы пацаны во дворе уважали

6 мин

26K

JavaScript*Node.JS*

Перевод

+12

Комментарии 13

maxp 4 мар 2019 в 11:31

Все эти сложности сделаны только для того, чтобы не пробрасывать явно конктекст обработки?

PrinceKorwin 4 мар 2019 в 20:38

Не всегда возможно явно пробрасывать контекст обработки (например используется сторонняя функция/библиотека).

aigoncharov 4 мар 2019 в 20:39

Если многоуровневые сервисы, то больно его руками передавать.
Все это для того, чтобы заменить Thread-local storage.

bocharovf 4 мар 2019 в 23:01

Лично меня чрезвычайно напрягает отсутствие каких-либо вменяемых зрелых стандартов по созданию trace ID

А OpenTracing чем не подошел?
Вот есть OpenTracing для Node.js
А вот Jaeger под ноду.

aigoncharov 4 мар 2019 в 23:30

Я обеими руками за opentracing, но не очень вижу как оно решает проблему создания trace ID автоматом.

serf 5 мар 2019 в 09:04

Возможно тупой вопрос. Для веба ведь есть Zone.js позволяющая «работать» с контекстом вызова, это нельзя прикрутить к ноде малой кровью?

mayorovp 5 мар 2019 в 09:11

Zone.js подменяет базовые асинхронные функции, такие как setTimeout. На ноде же их попросту слишком много.

А ещё Zone.js не умеет работать с функциями, объявленными как async, без транспиляции.

RidgeA 5 мар 2019 в 10:06

Все хорошо, конечно, но:

как у этого решения с производительностью?
Как-то не хочется в прод тащить экспериментальные модули (async_hooks), нет информации когда обещают перевести в статус стабильного?!

aigoncharov 5 мар 2019 в 14:36

1. На бенчмарки можно тут посмотреть github.com/nodejs/benchmarking/issues/181 и решить для себя насколько оно критично. Как по мне иметь возможность адекватной трассировки важнее.
2. Зависит от того, что у вас за прод. Меня, как любителя смузи и всего модного-стильного-молодежного, это не смутило и поюзать уже успел, но и компания была вполне себе хипстерская, а не кровавый энтерпрайз.

RidgeA 5 мар 2019 в 16:30

спасибо за ссылку, гляну.
но там только async_hooks, я помню касательно Proxy то же не все гладко (было по крайней мере, может сейчас это пофиксили)
Просто я бы это добавил в статью — как минимум то, что это эксперементальный модуль…

wentout 5 мар 2019 в 14:11

Все, кто сюда дочитал, обратите внимание ещё на такую вещь.

И cls-hooked и вообще всё, что основано на async_hooks имеет одну существенную проблему.
Проблема НЕ в асинхронной части. Она большей степени покрыта. Проблема в том, что синхронный контекст можно разделить. Например:

issues/59
issues/249
Пример для Front-End

Исходя из этого наивное применение подобных пакетов чревато потерей контекста.

Посему все нормальные пакеты рекомендуют делать обёртки в тех местах, которые не могут быть покрыты асинхронным контекстом автоматически. Поэтому «биндить» только req и res — мало. Надо биндить ещё и кучу всего остального, например общение с БД. Посмотреть можно, например, тут, самый конец файла: Cloud Tracing Mongoose Sync Split Wrapper: