-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
npm audit shows Vulnerabilites #93
Comments
Thanks for reporting this bug! =D It shouuuld be an easy fix, though I'm curious if I'm low on time today but I should be able to take a look by the end of next week (prob much sooner) |
It looks like we're on the latest There is a PR up for patching the But until that gets landed, I think we're stuck with your workaround for now =( Notes for self on how to reproduce:
|
Thank you for the feedback. Duly noted, wait and see :) |
@twolfson Now that get-pixels is using the native |
@ingalls Oh rad! I'll gladly update Unfortuantely, it looks like that while they landed the PR, there's no new release on npm? |
Partially related PR for this - #96. Unfortunately |
Thanks for the fix @ChrisdeWolf! It was too much overreach as a maintainer (e.g. my override can go out of date easily), but y'all can use it as a per-project override if you'd like: // Inside your package.json
"overrides": { "tough-cookie": "~4.1.3" }, I'll leave this issue open until |
6 moderate vulnerabilities exist in the current state of the package.
I was able to go down to 4 moderates by adding 2 overrides in my package.json:
"overrides": { "semver": "^6.3.1", "tough-cookie": "^4.1.3" }
The issues comes from the "request" package as shown below:
Is there a chance you can address these issues ?
Best regards
The text was updated successfully, but these errors were encountered: