search

winstonjs / node-loggly

A client implementation for Loggly in node.js
http://github.com/winstonjs/node-loggly
Other
233 stars 80 forks source link

Please update version number #26

Closed victorquinn closed 10 years ago

victorquinn commented 11 years ago

There was a pull request #14 which fixed an issue I'm having a few months ago.

However, it appears there hasn't been a bump in version number in the package.json for this module for almost 2 years: https://github.com/nodejitsu/node-loggly/commit/aa14779e642a611438f7341b03a1bc280c6922dc

This means that other modules, dependent on the change, haven't gotten any of the recent updates, including #14 . The issue specifically I ran into is that with https://github.com/indexzero/winston-loggly it's impossible to override the inputUrl.

However, the default url set in the 0.3.11 tagged version of this module (https://logs.loggly.com/input/...) is no longer valid. It will only work if I change that url (to https://logs-01.loggly.com/input/...) See http://loggly.com/docs/api-sending-data/

What this all means is that, with the default URL to loggly wrong and no way to override that, it's impossible to use winston-loggly or even node-loggly out of the box from npm.

This is very not ideal and burned a ton of my productivity today as I spent a long time trying to track down this issue which doesn't really have a good solution.

Thanks! Victor

nicco commented 11 years ago

pull request https://github.com/nodejitsu/node-loggly/pull/27

kb19 commented 11 years ago

+1 Burned tonnes of time today.

I posted my temporary solution here: https://github.com/nodejitsu/node-loggly/pull/27#issuecomment-25314557

nicco commented 11 years ago

I think the project is kind of dead, very annoying

indexzero commented 10 years ago

1.0.3 has been out for some time.