Bug 760286 - Repo urls with trailing whitespace are accepted but then fail to sync
Summary: Repo urls with trailing whitespace are accepted but then fail to sync
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Katello
Classification: Retired
Component: WebUI
Version: 1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Jason E. Rist
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: katello-sync-mgmt
TreeView+ depends on / blocked
 
Reported: 2011-12-05 18:54 UTC by Jeff Weiss
Modified: 2014-11-09 22:52 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-02-12 16:04:29 UTC
Embargoed:

Attachments (Terms of Use)

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.
Note You need to log in before you can comment on or make changes to this bug.