You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2020/03/19 13:18:56 UTC

[GitHub] [drill] ihuzenko commented on issue #2031: DRILL-7650: Add option to enable Jetty's dump for troubleshooting

ihuzenko commented on issue #2031: DRILL-7650: Add option to enable Jetty's dump for troubleshooting
URL: https://github.com/apache/drill/pull/2031#issuecomment-601172921
 
 
   > @ihuzenko, could you please explain, why `sslFactory.setProtocol(sslConf.getProtocol())` the call is excessive? I have looked into the `SslContextFactory` class, and the corresponding field is used in some places, for example in the `load()` method, and its non-default value is set only using its setter.
   
   According to [the doc ](https://www.eclipse.org/jetty/documentation/9.4.x/configuring-ssl.html#configuring-sslcontextfactory) it's enough to just set included protocols.  

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services