Bug 1086355

Summary: repository create form sending up empty string which pulp errors on as feed
Product: Red Hat Satellite Reporter: Tom McKay <tomckay>
Component: WebUIAssignee: Justin Sherrill <jsherril>
Status: CLOSED CURRENTRELEASE QA Contact: Katello QA List <katello-qa-list>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: NightlyCC: cwelton, jsherril, mmccune
Target Milestone: Unspecified   
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/5142
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-02 14:08:15 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tom McKay 2014-04-10 16:31:16 UTC
Page 11 Section 3.3 Step 6 - Clicking Save gives brief spinner but does not leave page. No error displayed in UI.

    The 'feed' form param is getting sent as "" (empty string). This is reproducible by entering a URL value in the form and then clearing it. (The step doesn't mention URL so I pasted the previous path there before deleting it before form submission. No error in UI. Can see error state in /foreman_tasks/dynflow (See @jsherrill for debug)

Comment 1 Tom McKay 2014-04-10 16:32:02 UTC
Created redmine issue http://projects.theforeman.org/issues/5142 from this bug

Comment 2 RHEL Program Management 2014-04-10 16:54:57 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 4 Justin Sherrill 2014-04-10 18:30:27 UTC
https://github.com/Katello/katello/pull/3969

Comment 5 Bryan Kearney 2014-04-23 11:39:49 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/5142 has been closed

Comment 7 Corey Welton 2014-05-06 20:46:08 UTC
Verified in Satellite-6.0.3-RHEL-6-20140430.4 -- however, I'm thinking we still should not be able to save a repo with no repourl, or not be able to initiate a sync on such a repo.  This is a separate bz I will enter afterwards.  However, for all intents and purposes, the behavior seen here is fixed.

Comment 8 Bryan Kearney 2014-07-02 14:08:15 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.

Comment 9 Bryan Kearney 2014-07-02 14:09:34 UTC
This was delivered in 6.0.3, the Beta version of Satellite 6.0