Github user ok2c commented on a diff in the pull request:

    https://github.com/apache/httpcomponents-core/pull/70#discussion_r202546439
  
    --- Diff: 
httpcore5-h2/src/main/java/org/apache/hc/core5/http2/impl/nio/ClientHttp2StreamMultiplexer.java
 ---
    @@ -83,10 +83,15 @@ public ClientHttp2StreamMultiplexer(
         Http2StreamHandler createRemotelyInitiatedStream(
                 final Http2StreamChannel channel,
                 final HttpProcessor httpProcessor,
    -            final BasicHttpConnectionMetrics connMetrics) throws 
IOException {
    +            final BasicHttpConnectionMetrics connMetrics,
    +            final Http2StreamHandler originalHandler) throws IOException {
             final HttpCoreContext context = HttpCoreContext.create();
             context.setAttribute(HttpCoreContext.SSL_SESSION, getSSLSession());
             context.setAttribute(HttpCoreContext.CONNECTION_ENDPOINT, 
getEndpointDetails());
    +        if (originalHandler != null && originalHandler instanceof 
ClientHttp2StreamHandler) {
    --- End diff --
    
    @ato I agree in principle but can this cast be avoided? 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to