Allow to configure readTimeout for HttpClient based McpClientTransport

[quaff]

Expected Behavior
The McpClientTransport builder should expose method to set read timeout of HTTP request, and requestTimeout of McpClient should be applied to its underlying McpClientTransport.

Current Behavior
HttpRequest is created by HttpClientSseClientTransport and HttpClientStreamableHttpTransport but without setting timeout.

Context
https://docs.oracle.com/en/java/javase/11/docs/api/java.net.http/java/net/http/HttpRequest.html#timeout()

The effect of not setting a timeout is the same as setting an infinite Duration, i.e. block forever.

[olamy]

You can use this:
sse

        HttpClientSseClientTransport transport = HttpClientSseClientTransport
                .builder("")
                .httpRequestCustomizer((builder, method, endpoint, body) -> builder.timeout(Duration.ofSeconds(30)))
                .build();  

streamable

        HttpClientStreamableHttpTransport transport = HttpClientStreamableHttpTransport
                .builder("")
                .httpRequestCustomizer((builder, method, endpoint1, body) -> builder.timeout(Duration.ofSeconds(30)))
                .build();

[quaff]

You can use this: sse

        HttpClientSseClientTransport transport = HttpClientSseClientTransport
                .builder("")
                .httpRequestCustomizer((builder, method, endpoint, body) -> builder.timeout(Duration.ofSeconds(30)))
                .build();  

streamable

        HttpClientStreamableHttpTransport transport = HttpClientStreamableHttpTransport
                .builder("")
                .httpRequestCustomizer((builder, method, endpoint1, body) -> builder.timeout(Duration.ofSeconds(30)))
                .build();

Yes, you could do that, but it's better to apply McpClient's requestTimeout to HttpRequest.Builder before httpRequestCustomizer.

[olamy]

sure you can some default using builder..requestBuilder(HttpRequest.newBuilder()timeout(Duration.ofSeconds(30))
it just depends on your need a default or a per-request timeout (depending on http method etc))
but it's possible