Red Hat Bugzilla – Full Text Bug Listing
|Summary:||spaces in URL crash pirut|
|Product:||[Fedora] Fedora||Reporter:||Allen <arouse>|
|Component:||pirut||Assignee:||Jeremy Katz <katzj>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Fedora Extras Quality Assurance <extras-qa>|
|Fixed In Version:||1.3.28||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2007-12-31 09:51:55 EST||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Bug Depends On:|
Description Allen 2007-11-09 19:52:30 EST
From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:18.104.22.168) Gecko/20071025 Firefox/22.214.171.124 Description of problem: A URL of the form file://<pathname with a space> will cause pirut to crash. Tried entering from the GUI when trying to add the dvd as a repo which was auto-mounted under /media/Fedora 8 DVD. So I tried to alter /etc/yum.repos.d/fedora.d (yes, I messed with a known good repo entry). Same result. It seems pirut expects all URLs to already be url-encoded. Had to remount dvd with a non-space containing pathname. Version-Release number of selected component (if applicable): pirut-1.3.25-2.fc8 How reproducible: Always Steps to Reproduce: 1. pick a <name>.repo file in /etc/yum.repos.d 2. add a space to the url of any baseurl definition 3. run pirut Actual Results: pirut shows a uninformative stack trace claiming that the url was "" and dies ... Expected Results: pirut understands urls with spaces Additional info:
Comment 1 Allen 2007-11-12 00:47:31 EST
The error is in yum, not pirut. The code causing this is /usr/lib/python2.5/site-packages/yum/config.py line 142. It is explicitly stated there that spaces are delimiters. So I suppose this is not a bug. Just not compliant with RFC 2396 or with all possible filenames. Perhaps some change to man yum.conf?
Comment 2 Jeremy Katz 2007-11-12 11:01:02 EST
Actually, we should do appropriate munging from, eg, space to %20 in pirut when doing the config editing. Will fix for an F8 update
Comment 3 Jeremy Katz 2007-11-19 14:18:33 EST
Fixed in git; will be pushed as an update shortly
Comment 4 Fedora Update System 2007-11-20 12:50:03 EST
pirut-1.3.26-1.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update pirut'
Comment 5 Fedora Update System 2007-11-28 20:34:02 EST
pirut-1.3.27-1.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update pirut'
Comment 6 Fedora Update System 2007-12-06 15:45:29 EST
pirut-1.3.28-1.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report.
Comment 7 Fedora Update System 2007-12-06 15:55:07 EST
pirut-1.3.28-1.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report.
Comment 8 Jeremy Katz 2007-12-31 09:51:55 EST
New versions of pirut have been released that should address these issues