Bug 140723
| Summary: | support for authenticated proxies has regressed | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Daniel Thompson <fedora> | ||||||
| Component: | python | Assignee: | Mihai Ibanescu <mihai.ibanescu> | ||||||
| Status: | CLOSED WONTFIX | QA Contact: | Brock Organ <borgan> | ||||||
| Severity: | medium | Docs Contact: | |||||||
| Priority: | medium | ||||||||
| Version: | 3 | CC: | katzj, mattdm | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | i386 | ||||||||
| OS: | Linux | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2006-07-11 19:26:24 UTC | Type: | --- | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Attachments: |
|
||||||||
|
Description
Daniel Thompson
2004-11-24 14:48:57 UTC
Created attachment 107390 [details]
Low quality band-aid patch to alleviate the problem
As already stated this avoids the problem rather then treating the cause.
If you get a chance- could you pull down yum from cvs and see if it helps out the problem you're having? Thanks. Sorry for the delay in replying. I've just downloaded a tarball from YUM viewcvs stuff (cannot use CVS directly from this machine). This did not work on FC3. Looking at the packet trace the problem of the extra new line is still apparant. Created attachment 118543 [details]
Remove a spurious newline from the authentication header
I finally got so fed up patching yum everytime it was updated I decided to find
the 'proper' fix for this problem.
It turns out to be a problem in Python's urllib2 library. The base64
encoder library included a newline on the end of the encoded string and this
was incorrectly being added to the header. A simple .rstrip() removed the
trailing whitespace and this fixes the broken header.
This non-hacky fix should be applied to Python.
Well, the even better fix would be to strip all newline characters in the string. encodestring has the bad habit of adding new lines every 80 chars (I believe). If the combination (username:password) happens to be longer than 58 chars, we're back to a similar problem. Fedora Core 3 is now maintained by the Fedora Legacy project for security updates only. If this problem is a security issue, please reopen and reassign to the Fedora Legacy product. If it is not a security issue and hasn't been resolved in the current FC5 updates or in the FC6 test release, reopen and change the version to match. Thank you! I believe this issue is fixed in recent Fedora Cores (and is certainly not a security issue). Thanks! |