Mantis - Squeak
Viewing Issue Advanced Details
912 Network trivial always 03-02-05 21:14 05-23-06 19:06
feedback 3.8  
0000912: Put proxy authentication back in HTTPSocket
Enable proxy authentication again in HTTPSocket Class >>#httpGetDocument:args:accept:request: after it was removed by change 6277HTTPSocketRedirectFix-st.

has duplicate 0007193new  Could not authenticate against a proxy/firewall... 
 HTTPAuthAgain-daf.1.cs [^] (4,306 bytes) 03-02-05 21:14
 HTTPAuthAgain2-daf.1.cs [^] (4,375 bytes) 04-20-05 16:01
 HTTPProxyCredentials.1.cs.gz [^] (1,964 bytes) 10-25-05 17:28
 ProxyAuthFix.1.cs.gz [^] (1,733 bytes) 03-02-06 23:38

03-08-05 13:12   
Monticello also includes a change which removes the proxy authentication, so it should be fixed, too.
03-08-05 16:57   
I know that the Monticello overlap was being worked on a while back, and thought it might be fixed by now. It is very possible to have both proxy authentication and server zone authentication in a single request.
04-20-05 16:02   
The newer attached file fixes this again after change 6651ExternalCleanup.
05-05-05 18:36   
The current version of Monticello on SqueakMap does indeed still contain a change that removes proxy authentication. I vaguely remember Avi saying something about being backward compatible so he couldn't remove the method from the changeSet, even though it wasn't really needed any more. Two points here: 1. proxy authentication is in fact in the current image and Monticello (mainly) is not. This is a bug fix to something in the image and should be put in. 2. Someone needs to figure out how to update Monticello so that it also includes this fix or doesn't include the method in question.
10-20-05 21:00   
Please review the current code in 3.9a-6693md5 (or later) it appears to me that proxy support is currently included and so this issue can be closed. Accurate?
10-22-05 11:37   
The problem is still there.
It's in the method HTTPSocket class>>httpGetDocument:args:accept:request:
In the request headers, it should include the contents of HTTPProxyCredentials when going through a proxy.
Since the header is ignored when not going through a proxy, it can be added in all cases.
10-25-05 17:30   
David and Hans-Martin,

I would really appreciate it if you could look at the HTTPProxyCredentials changeset I've attached to this issue. This is my attempt to adapt David's changes to what is currently in Squeak3.9a and I've made the use of the HTTPProxyCredentials contingent on actual use of the proxy server. If you can test this (this changeset is against 3.9a-6693md5) I would really appreciate it.
10-26-05 17:15   
The reformatting somewhat obfuscates the fact that this is basically a one line change, but yes, it works just fine. Thank you for your work on this issue!
03-02-06 23:37   
I see this issue is still waiting for feedback... the fix is working. Will it be integrated in 3.9 ?
I'm submitting the fix against 3.9a-7004
03-02-06 23:45   
The hold-up is that HTTPSocket is based on OldSocket which is destined for the chopping block, which may also be the destiny of HTTPSocket. The I/O Stewards team is still getting it's ducks in a row and I hope we can make some progress on this issue and related issues in the next few weeks.
05-23-06 19:06   
At this point it seems unlikely that we are going to do anything about OldSocket or changing HTTPSocket for 3.9.

So we need to finish up this issue. Can someone please try this in the most recent 3.9beta and confirm that it still does the desired job? I'm not behind a proxy server so I can't reliably test it.