Should McpTransportContextExtractor contain default methods for protocol version, lastEventId and sessionId

Offen

#492 erstellt am 14.8.2025 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: java
Bereich: api, backend-api-design

Rechercherichtung

Finde McpTransportContextExtractor und überprüfe seine bestehenden Implementierungen zusammen mit der Verwendung von McpTransportContext. Prüfe, ob Standardzugriffsmethoden für Protokollversion, Sitzungs-ID und ID des letzten Ereignisses zur aktuellen API und zu den Protokollkonzepten passen; abgeschlossen ist die Aufgabe, wenn die Schnittstelle eine konsistente, kompatible API bereitstellt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

waiting for triage

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.

Vorherrschende Sprache: Java
Sterne: 3.7k
Forks: 1.1k
Ø Merge: 1 T. 15 Std.
Gemergte PRs (30 T.): 9

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus modelcontextprotocol/java-sdk

SSE client silently truncates a data: line at U+2028/U+2029/U+0085, then fails with "Error parsing JSON-RPC message" Offen

area/transport bug P2

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

modelcontextprotocol/java-sdk#1136 · Erstellt am 18.9.2026
Client request handlers that complete empty send no JSON-RPC response Offen

area/client bug P2

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

modelcontextprotocol/java-sdk#1124 · 1 Kommentar · Erstellt am 03.9.2026
ServerCapabilities.logging is added unconditionally, overriding the caller's explicit capabilities Offen

bug P2 ready for work

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

modelcontextprotocol/java-sdk#1086 · 1 Kommentar · Erstellt am 11.8.2026
Reject listRoots if not supported by client, without sending any request Offen

enhancement good first issue P3

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

modelcontextprotocol/java-sdk#1067 · Erstellt am 19.7.2026
StdioClientTransport missing explicit UTF-8 charset in InputStreamReader (same issue as #295, but on client side) Offen

bug P2 ready for work

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100

modelcontextprotocol/java-sdk#898 · 1 Kommentar · Erstellt am 02.4.2026

Alle Issues in modelcontextprotocol/java-sdk