Why an empty callback is sent to winston.log?

[rictorres]

I just want to have a better understanding of this:

// This is fire and forget, we don't want logging to hold up the request so don't wait for the callback
options.winstonInstance.log('error', 'middlewareError', exceptionMeta, function () {
    // Nothing to do here
});

[floatingLomas]

It looks like it was added in late 2012; I don't actually think this is necessary, and since Winston would just skip calling the callback if it is not provided, it's probably introducing some unnecessary overhead.

[rictorres]

shall we get rid of it then? i already did it locally, gonna send a pr soon :)