Bug 760286

Summary:	Repo urls with trailing whitespace are accepted but then fail to sync
Product:	[Retired] Katello	Reporter:	Jeff Weiss <jweiss>
Component:	WebUI	Assignee:	Jason E. Rist <jrist>
Status:	CLOSED UPSTREAM	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.0	CC:	dajohnso, ehelms, mmccune
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-02-12 16:04:29 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:
Bug Depends On:
Bug Blocks:	767841

Description Jeff Weiss 2011-12-05 18:54:12 UTC

Description of problem:


Version-Release number of selected component (if applicable):
katello-0.1.123-1.git.15.658cadf.el6.x86_64

How reproducible:


Steps to Reproduce:
1. Create a repo with the following URL: "http://download.devel.redhat.com/released/F-16/GOLD/Fedora/x86_64/os/ " 
(include the trailing space but do not include quotes).
2. Sync the repo
3.
  
Actual results:
Creation of repo succeeds, but syncing of repo fails

Expected results:
Consistency - if the url is not automatically trimmed of whitespace, it should fail on create, not at sync time.  

I suggest a validation error at creation time: "Repo URL cannot contain leading or trailing whitespace"

Additional info:

Comment 1 Eric Helms 2013-02-12 16:04:29 UTC

This is fixed upstream and now throws a validation error.