Fix Timestamp Handling for Logstash and Default to True

[joshelson]

No description provided.

[cthayer]

Adding timestamps to the log event objects is something that winston does.

You can enable them by doing this:

var winston = require('winston'),
        LogstashUDP = require('winston-logstash-udp').LogstashUDP;

    var logger = new(winston.Logger)({
      transports: [
        new(LogstashUDP)({
          port: 9999,
          appName: 'my application name',
          host: '127.0.0.1',
          timestamp: true  // enable timestamps
        })
      ]
    });

It is intentional that winston's default behavior (no timestamps) is being preserved by this transport. Plus, I don't think that we should change the default behavior at this point as that breaks backwards compatibility.

[joshelson]

I don't believe the timestamp actually propagated with timestamp: true in that configuration. It was never passed up via the options stack.

Could certainly default it to false if that made more sense.