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

Consider changing dependencies #13

Closed
gonrial opened this issue Aug 14, 2017 · 6 comments
Closed

Consider changing dependencies #13

gonrial opened this issue Aug 14, 2017 · 6 comments

Comments

@gonrial
Copy link
Contributor

gonrial commented Aug 14, 2017

Please consider updating the dependencies. Right now we have node-uuid generating warnings: [SECURITY] node-uuid: crypto not usable, falling back to insecure Math.random()

Tracking back ... loggly package seems to be unmaintained (issue open five months back without answers). Loggly team created a fork repo to keep up security updates and created a new package.

@JorgenEvens
Copy link

We're also having issues due to this in our code.

@auberryberry
Copy link

+1 here would be great to swap dependencies

@bilby91
Copy link
Contributor

bilby91 commented Sep 29, 2017

Any updates ?

@MauriceButler
Copy link
Owner

Sorry everyone I have been away for a few weeks Ill get this sorted out now.

@auberryberry
Copy link

auberryberry commented Oct 3, 2017 via email

@MauriceButler
Copy link
Owner

FYI version 1.3.0 is now published which has been updated to use node-loggly-bulk.

This version should now install with no security warnings.

Due to some missing functionality in the node-loggly-bulk messages will no longer be sent to loggly in bulk.

As soon as the loggly/node-loggly-bulk#21 PR is merged I will release another version to bring back the bulk sending.

I will close this issue now and have raised #18 to track the change to re-enable bulk processing

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

No branches or pull requests

5 participants