DNS rebinding :* allowlist matches Host/Origin suffixes that are not ports

Aberta Para iniciantes

#3,463 aberta em 6/9/2026 2 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade: 2/5
Tempo estimado: 1-3 horas
Facilidade para iniciantes: 84/100

Tipo de issue: Bug
Clareza: Claramente especificada
Status de atividade: Ativa
Stack de tecnologia: python
Domínio: security

Direção de pesquisa

Comece em mcp/server/transport_security.py, onde TransportSecurityMiddleware compara allowed_hosts e allowed_origins, e depois leia tests/server/test_transport_security.py. Reproduza os casos de sufixo relatados e compare-os com a cobertura existente de portas numéricas. Está concluído quando as portas numéricas continuarem sendo aceitas, enquanto valores como wild.example:9000.evil e o Origin correspondente forem rejeitados.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

v1 v2

What happened

TransportSecurityMiddleware treats allowed_hosts / allowed_origins entries that end in :* as a prefix match: value.startswith(base + ":").

With allowed_hosts=["127.0.0.1:*"] or ["wild.example:*"], these Host values are accepted today:

127.0.0.1:8080.evil
wild.example:9000.evil

The same pattern accepts Origin http://wild.example:9000.evil for http://wild.example:*.

Existing tests only cover a numeric port (wild.example:9000). They do not cover a suffix after the port.

What I expected

base:* should mean base plus a numeric port, not any string that starts with base:.

How to reproduce

On main @ 08a3bc8:

from mcp.server.transport_security import TransportSecurityMiddleware, TransportSecuritySettings
from starlette.requests import Request

settings = TransportSecuritySettings(
    enable_dns_rebinding_protection=True,
    allowed_hosts=["wild.example:*"],
    allowed_origins=["http://wild.example:*"],
)
mw = TransportSecurityMiddleware(settings)
req = Request({"type": "http", "method": "GET", "headers": [(b"host", b"wild.example:9000.evil")]})
# validate_request returns None (accept). I expected 421.

I can send a PR that requires the suffix after base: to be digits, plus tests for the suffix cases. Happy to do that if you want it.

Written with AI assistance. I read the matcher next to tests/server/test_transport_security.py and reproduced it locally.

Linguagem predominante: Python
Estrelas: 24.3k
Forks: 4k
Merge médio: 1d 19min
PRs com merge (30d): 29

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

Leia a issue inteira e depois o guia de contribuição do projeto.
Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
Faça um fork do repositório e trabalhe em uma branch.
Abra um pull request que referencie o número da issue.