-
Notifications
You must be signed in to change notification settings - Fork 18
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
update package versions to fix windows-hosted instances #31
Conversation
I'm not too sure what snyk is on about, and I don't have access to it. |
Snyk is complaining about a vulnerable package in the dependency tree, try running |
Interesting!
|
Snyk please?
I even tried this, lol.
|
Attachments don't appear to have gone through the github notification system |
Okay. I'll do some research. |
isaacs/chownr#14 (comment) Somewhat concerning, and no real fix besides running containerized or with hardened permissions. |
@@ -397,7 +390,7 @@ | |||
}, | |||
"ws": { | |||
"version": "4.1.0", | |||
"resolved": "https://registry.npmjs.org/ws/-/ws-4.1.0.tgz", | |||
"resolved": "http://registry.npmjs.org/ws/-/ws-4.1.0.tgz", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay that's strange.
Fixes #30.
See Automattic/node-canvas#997 for details on the canvas constructor change.