[GitHub] [nifi] exceptionfactory commented on pull request #6048: NIFI-3869 Add HTTP/2 support to ListenHTTP and HandleHttpRequest

[GitBox <gi...@apache.org>]

exceptionfactory commented on PR #6048:
URL: https://github.com/apache/nifi/pull/6048#issuecomment-1132130583

   > I tested this using _openssl s_client -alpn h2 -connect 127.0.0.1:8101 -status_ and _openssl s_client -alpn http/1.1 -connect 127.0.0.1:8101 -status_ commands on both ListenHTTP and HandleHttpRequest and:
   > 
   >     * I was able to negotiate both ALPN protocols for when (h2 || http/1.1) was used
   > 
   >     * I was able to negotiate ALPN h2 for when (h2) was used
   > 
   >     * I was not able to negotiate ALPN for http/1.1 when http/1.1 was used
   > 
   > 
   > I'm not sure why the last case wasn't working.. probably not a big issue.
   
   Thanks for the feedback and testing @thenatog! For the last case, the `StandardServerConnectorFactory` does not add the ALPN Connector when `h2` is not selected. In other words, `http/1.1` indicates that ALPN will not be used, and the components will use standard TLS and HTTP/1.1 connectors.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org