Skip to content

[SECURITY] Status of DNS Rebinding Vulnerability Fix for rust-sdk #815

Closed

Description

@JLLeitschuh

I found (and reported to Anthropic here) that all major MCP Server SDKs were vulnerable to browser-based DNS rebinding allowing malicious websites to talk to locally or private network connected MCP servers.

This is was in violation of the MCP specification : https://modelcontextprotocol.io/specification/2025-06-18/basic/transports#security-warning

This resulted in a slew of CVEs assigned to every major SDK:

It's not clear to me what version, if it was at all, of the rust-sdk this was fixed in, as no CVE was assigned. Would the project maintainers behind this project be so kind as to do create a GitHub Security Advisory and request a CVE including the fix version and credit myself?

Thanks team!

Activity

  1. JLLeitschuh commented on Apr 17, 2026

    @JLLeitschuh
    Author
  2. added
    T-securitySecurity-related changes
    P0Critical: blocking, security, data loss, or crash
    on Apr 20, 2026
  3. DaleSeo commented on Apr 21, 2026

    @DaleSeo
    Member

    Hi @JLLeitschuh — thanks for the report and the follow-up.

    I believe the fix already landed in PR #764 and was first released in v1.4.0 on 2026-04-09.

    @alexhancock Do you want me to file a GitHub Security Advisory? I've never done it before but I can give it a try 😅

  4. JLLeitschuh commented on Apr 22, 2026

    @JLLeitschuh
    Author

    Let me know if you have any issues. Getting a CVE number is the best way to signal to your downstream users that this vulnerability was present and that they need to update their SDK version to adequately fix the issue. Without a CVE number, end-users are unlikely to update quickly

  5. DaleSeo commented on Apr 22, 2026

    @DaleSeo
    Member

    @JLLeitschuh @alexhancock The GitHub Security Advisory is drafted at GHSA-89vp-x53w-74fx. If you want any wording changed, the draft is editable, or leave comments here and I'll fold them in before publishing.

  6. JLLeitschuh commented on Apr 24, 2026

    @JLLeitschuh
    Author

    This looks good to me! 🎉 Great write up!

  7. alexhancock commented on Apr 24, 2026

    @alexhancock
    Contributor

    Thanks for doing it @DaleSeo

    How do we get a CVE number @JLLeitschuh ?

  8. JLLeitschuh commented on Apr 24, 2026

    @JLLeitschuh
    Author

    There should be a "Request CVE Number" down at the bottom, GitHub will issue you one

  9. alexhancock commented on Apr 24, 2026

    @alexhancock
    Contributor

    I requested it

    @DaleSeo Good to publish in your view?

    @JLLeitschuh Do you know if we should wait for the CVE to be issued before publishing or does it not matter?

  10. JLLeitschuh commented on Apr 24, 2026

    @JLLeitschuh
    Author

    Ideal to wait, it will probably happen by Monday

  11. DaleSeo commented on Apr 28, 2026

    @DaleSeo
    Member

    A CVE number hasn't been assigned yet. The advisory still shows cve_id: null as of today. I'll publish the advisory as soon as the CVE comes in, which will automatically close this issue.

  12. DaleSeo commented on Apr 29, 2026

    @DaleSeo
    Member

    The advisory has been published.

  13. removed
    P0Critical: blocking, security, data loss, or crash
    on Jul 30, 2026
  14. alexhancock commented on Jul 30, 2026

    @alexhancock
    Contributor

    Removing P0. This was advisory/CVE coordination, not critical-bug resolution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    T-securitySecurity-related changes

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions