Bug 720834

Summary:	[Providers Page] Repository URL needs to be trimmed
Product:	Red Hat Satellite	Reporter:	Jason E. Rist <jrist>
Component:	WebUI	Assignee:	Jason E. Rist <jrist>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.1	CC:	cwelton, mmccune
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-08-22 17:49:49 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:	747354

Description Jason E. Rist 2011-07-12 22:41:05 UTC

Description of problem:
Inputting a URL in the Repository URL with a trailing space leads to a space in the Repository URL in Pulp and Katello database.


Version-Release number of selected component (if applicable): 
988f5fbb69725c734208778e2f0f8094acf5a646

How reproducible:
very!


Steps to Reproduce:
1. clear all
2. start rails server
3. navigate to providers page:  https://0.0.0.0:3000/organizations/1/providers#panel=2
4. add space to a url at the end
5. save
6. upload subscription manifest
7. sync
8. list the pulp repos ('curl -k -u admin:admin https://localhost/pulp/api/repositories/ | json_reformat')
9. see space.
  
Actual results:
/var/lib/pulp//repos/pulp/repos/\ /content/dist/rhel/rhui/server/5Server/x86_64/rhui/1.2/os/

Expected results:
/var/lib/pulp//repos/pulp/repos/content/dist/rhel/rhui/server/5Server/x86_64/rhui/1.2/os/

Additional info:

Comment 1 Jason E. Rist 2011-07-19 04:26:23 UTC

Fixed in 16d88cd22f61e8849e15db95b9cf95a815bd9455

Added a before_validation filter.

Comment 2 Jason E. Rist 2011-07-20 14:41:36 UTC

8f8586eea88291b445effcef59ff034fc6ca247b Fixed.  Added a before_validation to strip trailing and preceding white space.

Comment 3 Jason E. Rist 2011-07-20 14:42:05 UTC

(In reply to comment #1)
> Fixed in 16d88cd22f61e8849e15db95b9cf95a815bd9455
> 
> Added a before_validation filter.

This was a bad push.  See https://bugzilla.redhat.com/show_bug.cgi?id=720834#c2

Comment 4 Corey Welton 2011-08-05 15:41:28 UTC

QA Verified, whitespace is stripped.

Comment 7 Mike McCune 2013-08-16 18:03:54 UTC

getting rid of 6.0.0 version since that doesn't exist