mitigate CVE-2025-62727 for starlette#1552
mitigate CVE-2025-62727 for starlette#1552nagmesh wants to merge 1 commit intomodelcontextprotocol:mainfrom
Conversation
There was a problem hiding this comment.
This CVE doesn't affect this SDK as we don't use the FileResponse object. For now I'm marking this as "Request changes" so we don't add such a strict minimum version and break a downstream applications/services.
Would be keen to hear what others think (especially @Kludex ), but I'm thinking we don't need to update this version as it doesn't affect the MCP Python SDK directly, and leave it up to users of this library to change the Starlette version used in their projects if needed.
|
This is not necessary. Exact same discussion on bump of vulnerable package versions: Kludex/uvicorn#2643 I've also reached a security expert, and this is not necessary, or wanted. |
|
@Kludex the vulnerable starlette version is pinned in the uv.lock and is causing downstream consumers to pull it in as a transitive dependency. Surely we aren't suggesting that everyone downstream of MCP should explicitly add the updated starlette version to resolve this since we're not willing to update the lock file, right? I agree with the non-pinned update in the pyproject.toml, but not updating the lock seems not ideal |
|
@ColeMurray Which downstream consumers rely on the |
Motivation and Context
The following dependencies contain High or Critical CVEs:
https://www.cve.org/CVERecord?id=CVE-2025-62727
How Has This Been Tested?
Package bumped to have version higher than 0.49.1
Breaking Changes
Types of changes
Checklist
Additional context