Message245582
There are two problems with the test case, and one bug in Python:
1. HTTPPasswordMgr doesn’t handle realm=None; it has to be a string. You can use HTTPPasswordMgrWithDefaultRealm though.
2. The password managers won’t match a proxy with a non-standard port number against a hostname without a port. So you have to include the port in the add_password() call.
3. AbstractDigestAuthHandler.get_authorization() is using the wrong URL, as Alexander already discovered. I made proxy-digest.patch which should fix this.
The wrong URL (final URL rather than proxy) was actually tested for in test_urllib2_localnet.ProxyAuthTests. So I fixed those tests. |
|
Date |
User |
Action |
Args |
2015-06-21 07:40:24 | martin.panter | set | recipients:
+ martin.panter, orsenthil, BreamoreBoy, toobaz, alexwe |
2015-06-21 07:40:24 | martin.panter | set | messageid: <1434872424.08.0.509563591039.issue16095@psf.upfronthosting.co.za> |
2015-06-21 07:40:23 | martin.panter | link | issue16095 messages |
2015-06-21 07:40:23 | martin.panter | create | |
|