Bug 720834 - [Providers Page] Repository URL needs to be trimmed
Summary: [Providers Page] Repository URL needs to be trimmed
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: Jason E. Rist
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: katello-blockers
TreeView+ depends on / blocked
 
Reported: 2011-07-12 22:41 UTC by Jason E. Rist
Modified: 2019-09-26 13:18 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 17:49:49 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.