bug: Кастомный логгер переопределяется не везде

[artems]

Bug Report

Environment and system information:

• ydb-nodejs-sdk version: 5.1.1
• Node.js version: 18.12.1
• NPM version: 8.19.2
• Operation System: Linux

Additional environment info:

Current behavior:

После вызова setupLogger sdk продолжает использовать FallbackLogger. Например, для сообщений вида:
Caught an error Overloaded, retrying with slow backoff, 10 retries left

Expected behavior:

Вызов setupLogger сразу после импорта sdk должен подменить logger для всех последующих logger.method() внутри sdk

Steps to reproduce:

При первом вызове setupLogger, после импорта sdk, в консоль всегда пишет предупреждение Reassigning logger, some logs can be lost. Это говорит о том, что FallbackLogger уже создан и где-то сохранен в переменную, которая не меняется после вызова setupLogger.

import { setupLogger } from 'ydb-sdk';
setupLogger(console);

Related code:

Внутри retryable создается инстанс RetryStrategy, в конструкторе которого вызывается getLogger, который создает FallbackLogger. FallbackLogger создается всегда, фактически, в момент первого подключения пакета ydb-sdk.

Некоторые методы продолжают использовать инстанс FallbackLogger, который получили в момент определения класса и вызова декоратора retryable для их методов, несмотря на последующий вызов setupLogger.

Other information:

Это релевантно этому issue:
#290