modelcontextprotocol / modelcontextprotocol/java-sdk
Should McpTransportContextExtractor contain default methods for protocol version, lastEventId and sessionId
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Java
- Estrelas
- 3.7k
- Forks
- 1.1k
- Merge médio
- 1d 15h
- PRs com merge (30d)
- 9
Descrição
I am defining my McpTransportContextExtractor implementation as:
@Internal
@Singleton
class DefaultMcpTransportContextExtractor implements McpTransportContextExtractor<HttpRequest<?>> {
public static final String HTTP_HEADER_MCP_PROTOCOL_VERSION = "MCP-Protocol-Version";
public static final String DEFAULT_PROTOCOL_VERSION = "2025-03-26";
public static final String HTTP_HEADER_MCP_SESSION_ID = "Mcp-Session-Id";
public static final String HTTP_HEADER_DEFAULT_LAST_EVENT_ID = "Last-Event-ID";
@Override
public McpTransportContext extract(HttpRequest<?> request, McpTransportContext transportContext) {
transportContext.put(HTTP_HEADER_MCP_PROTOCOL_VERSION,
request.getHeaders().get(HTTP_HEADER_MCP_PROTOCOL_VERSION, String.class)
.orElse(DEFAULT_PROTOCOL_VERSION));
request.getHeaders().get(HTTP_HEADER_MCP_SESSION_ID, String.class)
.ifPresent(v -> transportContext.put(HTTP_HEADER_MCP_SESSION_ID, v));
request.getHeaders().get(HTTP_HEADER_DEFAULT_LAST_EVENT_ID, String.class)
.ifPresent(v -> transportContext.put(HTTP_HEADER_DEFAULT_LAST_EVENT_ID, v));
return transportContext;
}
}
But given that the specification defines session id, last event and protocol version. I think it would be good if the McpTransportContextExtractor contained default methods:
default String getSessionId() {
return null;
}
default String getLastEventId() {
return null;
}
default String getProtocolVersion() {
return "2025-03-26";
}
I can submit a PR if you think it is a good idea. With this change the TransportContext will have a clear API for those spec defined concepts.
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.
Direção de pesquisa
Localize McpTransportContextExtractor e revise suas implementações existentes juntamente com o uso de McpTransportContext. Verifique se os acessadores padrão para a versão do protocolo, o ID da sessão e o ID do último evento são compatíveis com a API atual e com os conceitos do protocolo; considera-se concluído quando a interface expõe uma API consistente e compatível.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- java
- Domínio
- api, backend-api-design
- Tipo de issue
- Funcionalidade
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 45/100