search

loggly / node-loggly-bulk

A client implementation for Loggly in node.js
https://www.loggly.com/docs/node-js-logs/
Other
20 stars 20 forks source link

Allow bulk messages to be passed as an array #21

Closed MauriceButler closed 7 years ago

MauriceButler commented 7 years ago

This PR allows for the previous bulk logging functionality from winstonjs/node-loggly which is relied on by bunyan-loggly.

The PR will allow bunyan-loggly to switch from winstonjs/node-loggly to node-loggly-bulk as per https://github.com/MauriceButler/bunyan-loggly/issues/13

mostlyjason commented 7 years ago

@MauriceButler thanks for the effort you put into this! I have added a task to our team's backlog to test this. We have a couple issues in the queue ahead of this one but we should be able to test it in the next couple weeks.

Shwetajain148 commented 7 years ago

@mchaudhary @mostlyjason I have tested it. The PR seems good to me.