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

Express dependency needs upgrade in @nestjs/platform-express #14061

Open
3 of 15 tasks
mo-alaa opened this issue Oct 10, 2024 · 1 comment
Open
3 of 15 tasks

Express dependency needs upgrade in @nestjs/platform-express #14061

mo-alaa opened this issue Oct 10, 2024 · 1 comment
Labels
needs triage This issue has not been looked into

Comments

@mo-alaa
Copy link

mo-alaa commented Oct 10, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current behavior

The express dependency depends on the cookie package which has a security vulnerability: CVE-2024-47764
This is solved in cookie version 0.7.0, i.e. express upgrade to the latest version: 4.21.1 which is why this dependency needs updating.

Minimum reproduction code

https://www.mend.io/vulnerability-database/CVE-2024-47764

Steps to reproduce

No response

Expected behavior

According to your policy security vulnerabilities should be sent via email, but I sent an email but got no response. That's why I opened this ticket.

Thank you :)

Package

  • I don't know. Or some 3rd-party package
  • @nestjs/common
  • @nestjs/core
  • @nestjs/microservices
  • @nestjs/platform-express
  • @nestjs/platform-fastify
  • @nestjs/platform-socket.io
  • @nestjs/platform-ws
  • @nestjs/testing
  • @nestjs/websockets
  • Other (see below)

Other package

No response

NestJS version

10.4.4

Packages versions

"@nestjs/common": "^10.4.4",
"@nestjs/core": "^10.4.4",
"@nestjs/platform-express": "^10.4.4",

Node.js version

No response

In which operating systems have you tested?

  • macOS
  • Windows
  • Linux

Other

No response

@mo-alaa mo-alaa added the needs triage This issue has not been looked into label Oct 10, 2024
@ezintz
Copy link

ezintz commented Oct 10, 2024

Hello, I already created a pull request for those: #14060

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage This issue has not been looked into
Projects
None yet
Development

No branches or pull requests

2 participants