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.

Sign up for GitHub

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

Jump to bottom

v1.2.0 is failing NSP security check #17

Closed
marin-liovic opened this issue Oct 3, 2017 · 3 comments · Fixed by #21
Closed

v1.2.0 is failing NSP security check #17

marin-liovic opened this issue Oct 3, 2017 · 3 comments · Fixed by #21

Comments

@marin-liovic
Copy link

This is failing NSP check because of a dependency it uses. More information here: https://nodesecurity.io/advisories/533
Since dependency might not get patched soon, is there a possibility to remove it from this project or use a secure one?
@MauriceButler
Copy link
Owner

MauriceButler commented Oct 3, 2017 •
edited
Loading

Hi @marin-liovic,

I just released version 1.3.0 a few minutes ago which addresses this issue as per #13

I was incorrect the timespan issue is still present.

The downsteam issue is here loggly/node-loggly-bulk#20

@MauriceButler MauriceButler reopened this Oct 3, 2017
@mostlyjason
Copy link

The above issue has been fixed

@MauriceButler MauriceButler mentioned this issue Oct 31, 2017
Merged
@MauriceButler
Copy link
Owner

Version v1.3.2 has been published which now passes NSP check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

upgraded to latest node-loggly-bulk to fix nsp error MauriceButler/bunyan-loggly
3 participants
@MauriceButler @marin-liovic @mostlyjason