Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue using winston-loggly-bulk with winston 2.3.x #10

Closed
sloops77 opened this issue Feb 2, 2017 · 3 comments
Closed

Issue using winston-loggly-bulk with winston 2.3.x #10

sloops77 opened this issue Feb 2, 2017 · 3 comments

Comments

@sloops77
Copy link

@sloops77 sloops77 commented Feb 2, 2017

See winstonjs/winston#947

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Jun 6, 2017

@sloops77, I setup Winston and winston-loggly-bulk on my machine and tried to reproduce this issue at my end but I couldn't see any error as mentioned in above link. I also setup Winston v2.3.0and v2.3.1(because you mentioned "Issue using winston-loggly-bulk with Winston v2.3.x") with different node versions like- 4.5.0, 6.10.1 and 7.7.2 but didn't face any issue.

I am attaching my code snippet that I tried with and what standard format loggly suggests to log any string or object when you are using winston-loggly-bulk with Winston. Please refer the below code:

var winston = require('winston');
require('winston-loggly-bulk');

winston.add(winston.transports.Loggly, {
    inputToken: "token",
    subdomain: "subdomain",
    tags: ["winston"],
    json: true
    
});

 winston.log('info', 'hey there');

It would be a great help if you can share your code snippet and environment details(like node's version) in which you are seeing this issue.

cc: @mostlyjason

@sloops77
Copy link
Author

@sloops77 sloops77 commented Jun 28, 2017

Hi @Shwetajain148
will take a look next and get back to you.

@mostlyjason
Copy link

@mostlyjason mostlyjason commented Jul 18, 2017

This should be fixed now so we are closing the issue. Reopen if it still happens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.