Flushing %d events

[tommoor]

Describe the bug
The SDK constantly prints "Flushing %d events" to the console.

Expected behavior

• Fix to properly interpolate the count, don't print anything if nothing was flushed, or just remove this altogether?

Additional context

node-server-sdk/event_processor.js

Line 217 in 7098956

config.logger.debug('Flushing %d events', worklist.length);

[kinyoklion]

Hello @tommoor,

Are you using a custom logger with your SDK? The basic logger uses line = util.format(...tempArgs); to interpolate log lines.

Here is some output using the basicLogger provided with the SDK. Adjusted to show debug level items.

debug: [LaunchDarkly] Flushing 21 events

The creation of the logger:

 basicLogger({
 level: 'debug',
 });

You can also use the basic logger and change the destination.

basicLogger({
 level: 'debug',
 destination: line => {
 console.log(line);
 },
 });

If you are directly implementing the LDLogger interface, then it receives variadic arguments and you need to interpolate them.

The first of the args will be the format string, then any additional args.

 logger: {
 debug: (...args) => {
 console.log(...args);
 },
 info: (...args) => {
 console.log(...args);
 },
 warn: (...args) => {
 console.log(...args);
 },
 error: (...args) => {
 console.log(...args);
 },
 },

Produces:

Flushing 2 events

Thank you!
Ryan

[tommoor]

Thanks, looks like this is an issue with our implementation of the interface then 👍