[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-2417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18067117#comment-18067117
 ] 

ASF subversion and git services commented on HTTPCLIENT-2417:
-------------------------------------------------------------

Commit 30386d309614309b18e7b23953ee42bded18d57b in httpcomponents-client's 
branch refs/heads/5.6.x from Arturo Bernal
[ https://gitbox.apache.org/repos/asf?p=httpcomponents-client.git;h=30386d309 ]

HTTPCLIENT-2417 Honor TlsConfig attachment in async connect path

Make PoolingAsyncClientConnectionManager.connect() use the provided TlsConfig 
for direct TLS connections.

(cherry picked from commit df4347f34528dd4b10b2aef968086dd589d99428)


> HttpAsyncClientBuilder.setVersionPolicy() has no effect on 
> PoolingAsyncClientConnectionManager.connect()
> --------------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-2417
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-2417
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient (async)
>    Affects Versions: 5.6
>            Reporter: Laura Trotta
>            Priority: Major
>             Fix For: 5.6.1, 5.7-alpha1
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> Verified in httpclient5 5.4.4 and 5.6, probably present in previous versions. 
> {*}HttpAsyncClientBuilder{*}.setVersionPolicy() sets the *tlsConfig* object 
> in the builder, which is then passed to {*}InternalHttpAsyncClient{*}, then 
> {*}InternalHttpAsyncExecRuntime{*}, and finally to 
> {*}PoolingAsyncClientConnectionManager{*}'s connect() and upgrade() as an 
> Object (the *attachment* arg).  While upgrade() checks *attachment* before 
> calling {*}resolveTlsConfig{*}, connect() ignores it.
> This results in {*}HttpAsyncClientBuilder{*}.setVersionPolicy() doing 
> effectively nothing in case of direct connections. The method is deprecated 
> and there's a working alternative (using .setDefaultTlsConfig() in 
> {*}PoolingAsyncClientConnectionManagerBuilder{*}), but it's still documented 
> as functional, and confusing for users. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to