Skip to content

Peer dependency vulnerability : express 5.1.0 #462

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

Open
dacloutier-logmein opened this issue May 8, 2025 · 0 comments
Open

Peer dependency vulnerability : express 5.1.0 #462

dacloutier-logmein opened this issue May 8, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@dacloutier-logmein
Copy link

Describe the bug
Our CI scans raised a potential vulnerability via peer dependency express 5.1.0

citing the following issues:
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2024-10491
https://cwe.mitre.org/data/definitions/23.html

Image

To Reproduce
Steps to reproduce the behavior:

  1. execute a vulnerability scan (this was caught as part of our CI process)

Expected behavior
Try to not have potential peer dependency vulnerability, if it can be avoided.

Logs
see screenshot attached.

Additional context
By reverting to a previous version of eslint that was not pulling on mcp-sdk, the issue isn't present.

➜ pnpm why express 5.1.0
Legend: production dependency, optional only, dev only

devDependencies:
eslint 9.26.0
└─┬ @modelcontextprotocol/sdk 1.11.0
├── express 5.1.0
└─┬ express-rate-limit 7.5.0
└── express 5.1.0 peer
eslint-config-prettier 10.1.2
└─┬ eslint 9.26.0 peer
└─┬ @modelcontextprotocol/sdk 1.11.0
├── express 5.1.0
└─┬ express-rate-limit 7.5.0
└── express 5.1.0 peer
eslint-plugin-playwright 2.2.0
└─┬ eslint 9.26.0 peer
└─┬ @modelcontextprotocol/sdk 1.11.0
├── express 5.1.0
└─┬ express-rate-limit 7.5.0
└── express 5.1.0 peer
typescript-eslint 8.32.0
├─┬ @typescript-eslint/eslint-plugin 8.32.0
│ ├─┬ @typescript-eslint/parser 8.32.0 peer
│ │ └─┬ eslint 9.26.0 peer
│ │ └─┬ @modelcontextprotocol/sdk 1.11.0
│ │ ├── express 5.1.0
│ │ └─┬ express-rate-limit 7.5.0
│ │ └── express 5.1.0 peer
│ ├─┬ @typescript-eslint/type-utils 8.32.0
│ │ ├─┬ @typescript-eslint/utils 8.32.0
│ │ │ └─┬ eslint 9.26.0 peer
│ │ │ └─┬ @modelcontextprotocol/sdk 1.11.0
│ │ │ ├── express 5.1.0
│ │ │ └─┬ express-rate-limit 7.5.0
│ │ │ └── express 5.1.0 peer
│ │ └─┬ eslint 9.26.0 peer
│ │ └─┬ @modelcontextprotocol/sdk 1.11.0
│ │ ├── express 5.1.0
│ │ └─┬ express-rate-limit 7.5.0
│ │ └── express 5.1.0 peer
│ ├─┬ @typescript-eslint/utils 8.32.0
│ │ └─┬ eslint 9.26.0 peer
│ │ └─┬ @modelcontextprotocol/sdk 1.11.0
│ │ ├── express 5.1.0
│ │ └─┬ express-rate-limit 7.5.0
│ │ └── express 5.1.0 peer
│ └─┬ eslint 9.26.0 peer
│ └─┬ @modelcontextprotocol/sdk 1.11.0
│ ├── express 5.1.0
│ └─┬ express-rate-limit 7.5.0
│ └── express 5.1.0 peer
├─┬ @typescript-eslint/parser 8.32.0
│ └─┬ eslint 9.26.0 peer
│ └─┬ @modelcontextprotocol/sdk 1.11.0
│ ├── express 5.1.0
│ └─┬ express-rate-limit 7.5.0
│ └── express 5.1.0 peer
├─┬ @typescript-eslint/utils 8.32.0
│ └─┬ eslint 9.26.0 peer
│ └─┬ @modelcontextprotocol/sdk 1.11.0
│ ├── express 5.1.0
│ └─┬ express-rate-limit 7.5.0
│ └── express 5.1.0 peer
└─┬ eslint 9.26.0 peer
└─┬ @modelcontextprotocol/sdk 1.11.0
├── express 5.1.0
└─┬ express-rate-limit 7.5.0
└── express 5.1.0 peer

@dacloutier-logmein dacloutier-logmein added the bug Something isn't working label May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant