-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Heads up of Node.js security releases 12th October 2021 #1574
Comments
@mcollina do you think CVE-2021-3807 would be patched for |
Probably not, it's pulled in transitively by npm - you should ask them to update whatever is needed down their dependency tree, then that will trickle into the node distro and then into the docker image. $ docker run -it --entrypoint=sh node:alpine -c 'npm ls -g ansi-regex'
/usr/local/lib
`-- npm@7.24.0
+-- cli-columns@3.1.2
| +-- string-width@2.1.1
| | `-- strip-ansi@4.0.0
| | `-- ansi-regex@3.0.0
| `-- strip-ansi@3.0.1
| `-- ansi-regex@2.1.1
`-- cli-table3@0.6.0
`-- string-width@4.2.2
`-- strip-ansi@6.0.0
`-- ansi-regex@5.0.0 npm/cli#3813 was included in yesterday's v8 release of npm, so you're probably out of luck though (I don't know this). And regardless, $ docker run -it --entrypoint=sh node:alpine -c 'npm install -g npm && npm ls -g ansi-regex'
removed 44 packages, changed 24 packages, and audited 221 packages in 8s
10 packages are looking for funding
run `npm fund` for details
3 moderate severity vulnerabilities
To address all issues, run:
npm audit fix
Run `npm audit` for details.
npm notice
npm notice New major version of npm available! 7.24.0 -> 8.0.0
npm notice Changelog: /~https://github.com/npm/cli/releases/tag/v8.0.0
npm notice Run npm install -g npm@8.0.0 to update!
npm notice
/usr/local/lib
`-- npm@8.0.0
+-- cli-columns@4.0.0
| `-- strip-ansi@6.0.1
| `-- ansi-regex@5.0.1
+-- cli-table3@0.6.0
| `-- string-width@4.2.2
| `-- strip-ansi@6.0.0
| `-- ansi-regex@5.0.0
+-- columnify@1.5.4
| `-- strip-ansi@3.0.1
| `-- ansi-regex@2.1.1
`-- npmlog@5.0.1
`-- gauge@3.0.1
`-- string-width@2.1.1
`-- strip-ansi@4.0.0
`-- ansi-regex@3.0.0 In summary - you need to ask At time of writing the latest commit on Node's master branch is an update to |
Release is out, but the MUSL builds aren't ready. I'll be AFK the rest of the evening, /cc @nodejs/docker for others to make the update 🙂 |
Yeah I'm also looking into this, the unofficial is always a blocker to this 😢
|
The releases are out ad ready for integration. |
No, not the musl ones. https://unofficial-builds.nodejs.org/download/release/ |
16 is out, just 14 & 12 to go, but it's moving |
Upstream PR docker-library/official-images#11094 |
Hello.
|
I'm seeing what @f2404 is seeing too. Is there something I'm doing wrong when using the container? Is this not the correct container to be using for the base in the dockerfile? |
See #1574 (comment). This is npm, not the docker image. Report it to npm |
As per the Node.js security release process this is the FYI that there is going to be a security release 12th October 2021
The text was updated successfully, but these errors were encountered: