[ 
https://issues.apache.org/jira/browse/GERONIMO-3706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12553452
 ] 

Sangjin Lee commented on GERONIMO-3706:
---------------------------------------

I was able to write a quick proof-of-concept that enables proxy support 
including tunneling for https.  I'm attaching the patch *strictly for your 
review*.  It's not polished, and is not meant to be applied directly.

It has very little comments, may not handle all the edge cases like connection 
retries, timeouts, error cases, etc.  Also, it contains a unit test, but it 
requires a presence of a proxy at localhost:8888, so it probably shouldn't be 
added permanently...

I was able to confirm that it works for http as well as https.  I used Charles 
to test it.

One other aspect is whether we want to enable connection reuse for proxied 
connections.  I think it's a decision we need to make one way or the other...  
The POC ignores connection reuse.



> support for proxy
> -----------------
>
>                 Key: GERONIMO-3706
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-3706
>             Project: Geronimo
>          Issue Type: New Feature
>      Security Level: public(Regular issues) 
>          Components: AsyncHttpClient
>    Affects Versions: 1.x
>            Reporter: Sangjin Lee
>
> Proxy support is a critical feature for HTTP clients.  I'd like to have 
> AsyncHttpClient support proxy.  The following would be considered as the 
> basic features:
> - Enabling connecting through proxies for http and https targets
> - Exclusion (domains that should not go through proxies)
> - Allowing proxy related configuration on AsyncHttpClient
> - Support for proxy authentication, at least for Basic authentication (and 
> perhaps Digest too?)
> There are things like SOCKS support, etc., but the above will be a good 
> start.  Thoughts?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to