G2 Loggly support only

[indexzero]

Will be v1.0.0

cc/ @dtudury @ltudury

[ronkorving]

Awesome work, looking forward to this one :+1:

[keithamus]

When can we expect this to drop?

[joeybaker]

Need any help with this? I just got upgraded and am looking to use this as soon as possible.

[jcrugzz]

@joeybaker @keithamus @ronkorving Hey guys, I'd encourage all of you guys to just give it a try npm install "git://github.com/nodejitsu/node-loggly.git#v1.0.0-dev" and report back with what you find :).

[joeybaker]

@jcrugzz Cool, so things should be working? I was not able to get logs to propagate to loggly but all I did saw install winston-loggly#1.0.0-dev into a current project and pray. :) Zero debuggin' on my part, I just figured things were in a known broken state still… apparently not though?

[jcrugzz]

@joeybaker I haven't been able to test it at all personally, this PR is essentially up for review. There may be some minor bugs but it should be in a reasonable state.

[topliceanu]

this PR is essentially up for review

How can I help? Is there a roadmap for the v1 release?

[travispbrown]

+1 for someone patiently waiting on this. We can't upgrade to Gen 2 until we have this in place. Let me know if I can help in any way.

[keithamus]

Just as a quick fix, you can actually do the following to replace the URL to Loggly, where you instantiate your config:

var client = loggly.createClient({
    subdomain: 'mydomain',
    auth: null,
    json: true,
});
Object.defineProperty(client.config, 'inputUrl', {
    value: 'https://logs-01.loggly.com/inputs/',
    enumerable: true,
    configurable: true,
});

If you're using winston-loggly, this just extends to:

var loggly = new (winston.transports.Loggly)({
    inputToken: 'mytoken',
    subdomain: 'mydomain',
    json: true,
});
Object.defineProperty(loggly.client.config, 'inputUrl', {
    value: 'https://logs-01.loggly.com/inputs/',
    enumerable: true,
    configurable: true,
});

Also you can cheat with the input token by doing inputToken += '/tags/foo,bar' to tag all logs with foo and bar

[meatballhat]

:thumbsup: adding myself to the list of folks hoping this'll drop soon :smiley_cat:

[aj0strow]

@keithamus thank you!

[arsnl]

Thanks @keithamus ! Nice quick fix until the new release :)

[fizerkhan]

+1. Waiting for the release

[nickpoorman]

+1

[julienandre]

+1

[panzerdp]

Thanks a lot!

[brandoncarl]

FANTASTIC

[fizerkhan]

Thanks mate.

[julienandre]

Works great, Thanks !!

[circuitbomb]

+1

[jcrugzz]

@circuitbomb this is supported in 1.0.x ;).

[circuitbomb]

Yea, looking at winston-loggly's dep. it's using 1.0.x but for some reason it still wasn't pushing my logs there until i included @keithamus tempfix, so in that regard it's either winston-loggly or the way I'm implementing the transport in my app.