winstonjs / winston-loggly

A Loggly transport for winston

Home Page:http://github.com/indexzero/winston-loggly

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Handle loggly errors

naholyr opened this issue · comments

Currently, there is absolutely no error handling:

    this.client.log(this.inputToken, message, function () {
      self.emit('logged');
      callback(null, true);
    });

Today I face (not sure if it's usual with Loggly, I use it only very recently) a lot of "Error: socket hang up" errors. At least half of my events are not sent, and there is no way I can know it from the transport :(

You could change this part to something like this so we can plug to "error" event:

    this.client.log(this.inputToken, message, function (err) {
      self.emit(err ? 'error' : 'logged', err);
      callback(err, !!err);
    });

Or at least pass err to callback so we can catch it and re-call log().

+1 to @naholyr , 4 years passed but this code is still in the repo and being a real pain.