[
https://issues.apache.org/jira/browse/HTTPCLIENT-669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514755
]
Roland Weber commented on HTTPCLIENT-669:
-----------------------------------------
Hi Andrea,
yes, you've definitely got the spirit right :-) Thanks a lot.
If you could make one more tweak to it: instead of passing the planner to the
constructor of the HttpClient implementations, could you use the default
instantiation strategy that Oleg has implemented for the other helper objects?
I'm about to wrap it up for today and will be offline tomorrow evening. If Oleg
doesn't check it in, I'll do it later this week.
This issue should remain open though, as there are a fews things left to figure
out. In particular, I don't like the dependency of the route planner on the
connection manager. It needs the scheme registry though. I'm afraid there might
be a bigger design problem lurking behind.
thanks again,
Roland
> introduce HttpRoutePlanner interface
> ------------------------------------
>
> Key: HTTPCLIENT-669
> URL: https://issues.apache.org/jira/browse/HTTPCLIENT-669
> Project: HttpComponents HttpClient
> Issue Type: Improvement
> Components: HttpClient, HttpConn
> Reporter: Roland Weber
> Fix For: 4.0 Alpha 2
>
> Attachments: firstImpl.patch
>
>
> Define an interface to determine a route for a given target host.
> Create default implementation replacing DefaultHttpClient.determineRoute(...);
> Implementations will need access to params and/or request.
> The interface fits into HttpConn, but DHC.dR(...) uses client parameters.
> Either move parameters to HttpConn, or keep default implementation in
> HttpClient.
> Alternative implementations could evaluate Java system properties related to
> proxy settings.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
