Bug 720834

Summary: [Providers Page] Repository URL needs to be trimmed
Product: Red Hat Satellite Reporter: Jason E. Rist <jrist>
Component: WebUIAssignee: Jason E. Rist <jrist>
Status: CLOSED CURRENTRELEASE QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.1CC: cwelton, mmccune
Target Milestone: UnspecifiedKeywords: 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:
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