[
https://issues.apache.org/jira/browse/HTTPCLIENT-2209?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andriy Redko updated HTTPCLIENT-2209:
-------------------------------------
Description:
Although the *AsyncClientConnectionOperator* is primarily used by
*AsyncClientConnectionManager* implementation, the *HttpContext* is not passed
from *AsyncClientConnectionManager* to *AsyncClientConnectionOperator* (the
respective connect() and upgrade() methods just do nothing with it).
It significantly limits the context propagation abilities for
*AsyncClientConnectionOperator* implementations. Also, adding *HttpContext*
makes the *AsyncClientConnectionOperator* on par with
*HttpClientConnectionOperator* which accepts HttpContext.
The change could be done in non-breaking fashion by adding default method
implementation. Happily to submit a patch if the improvement makes sense.
was:
Although the *AsyncClientConnectionOperator* is primarily used by
*AsyncClientConnectionManager* implementation, the *HttpContext* is not passed
from *AsyncClientConnectionManager* to *AsyncClientConnectionOperator* (the
respective connect() and upgrade() methods just do nothing with it).
It significantly limits the context propagation abilities to
*AsyncClientConnectionOperator* implementations. Also, adding *HttpContext*
makes the *AsyncClientConnectionOperator* on par with
*HttpClientConnectionOperator* which accepts HttpContext.
The change could be done in non-breaking fashion by adding default method
implementation. Happily to submit a patch if the improvement makes sense.
> Pass HttpContext to AsyncClientConnectionOperator
> -------------------------------------------------
>
> Key: HTTPCLIENT-2209
> URL: https://issues.apache.org/jira/browse/HTTPCLIENT-2209
> Project: HttpComponents HttpClient
> Issue Type: Improvement
> Components: HttpClient (async)
> Affects Versions: 5.1.2
> Reporter: Andriy Redko
> Priority: Major
>
> Although the *AsyncClientConnectionOperator* is primarily used by
> *AsyncClientConnectionManager* implementation, the *HttpContext* is not
> passed from *AsyncClientConnectionManager* to *AsyncClientConnectionOperator*
> (the respective connect() and upgrade() methods just do nothing with it).
> It significantly limits the context propagation abilities for
> *AsyncClientConnectionOperator* implementations. Also, adding *HttpContext*
> makes the *AsyncClientConnectionOperator* on par with
> *HttpClientConnectionOperator* which accepts HttpContext.
> The change could be done in non-breaking fashion by adding default method
> implementation. Happily to submit a patch if the improvement makes sense.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
