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

Update Docker base images to remove high severity vulnerability #1627

Closed
asos-robbell opened this issue Aug 25, 2023 · 3 comments
Closed

Update Docker base images to remove high severity vulnerability #1627

asos-robbell opened this issue Aug 25, 2023 · 3 comments
Labels
🐋 docker Related to Docker code 🔒 security Address a security vulnerability

Comments

@asos-robbell
Copy link

Node 18.16.1 includes a high severity vulnerability reported by Snyk. Updating base images to 18.17.1 removes this vulnerability.

https://security.snyk.io/vuln/SNYK-UPSTREAM-NODE-5848038

Happy to open a PR for this if that helps.

@ggrossetie
Copy link
Member

I believe there's already one PR: #1596

According to the CVE this vulnerability affects process.binding():

Affected versions of this package are vulnerable to Arbitrary Code Injection. The use of the deprecated API process.binding() can bypass the policy mechanism by requiring internal modules and eventually take advantage of process.binding('spawn_sync') run arbitrary code, outside of the limits defined in a policy.json file.

https://security.snyk.io/vuln/SNYK-UPSTREAM-NODE-5848038

Since we are not using process.binding() I think we are pretty safe.

@ggrossetie ggrossetie added 🐋 docker Related to Docker code 🔒 security Address a security vulnerability labels Aug 28, 2023
@heyLu
Copy link
Contributor

heyLu commented Jan 9, 2024

This issue can probably closed as the PR with the update to 18.17 (#1596) has been merged?

@ggrossetie
Copy link
Member

Indeed, closing 🚪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐋 docker Related to Docker code 🔒 security Address a security vulnerability
Projects
None yet
Development

No branches or pull requests

3 participants