Bug 1099221

Summary:	CDN path verification should handle redirects
Product:	Red Hat Satellite	Reporter:	Erik M Jacobs <ejacobs>
Component:	WebUI	Assignee:	Ivan Necas <inecas>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Tazim Kolhar <tkolhar>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.3	CC:	cwelton, inecas, jmontleo, mmccune, omaciel, tkolhar
Target Milestone:	Unspecified	Keywords:	Reopened, Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/5574
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-07-02 14:14:41 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 Erik M Jacobs 2014-05-19 19:15:20 UTC

Description of problem:
Currently the cdn_var_substitutor#valid_substitutions? treats 301 redirects as an error. In some cases it will try to fetch $REPO_URL/repodata which causes a redirect to $REPO_URL/repodata/. One solution is to have it request $REPO_URL/repodata/repomd.xml, but i would argue it should just follow redirects as well.

Version-Release number of selected component (if applicable):
katello-1.5.0-22.el6sat.noarch
katello-ca-1.0-1.noarch
katello-certs-tools-1.5.5-1.el6sat.noarch
katello-installer-0.0.37-1.el6sat.noarch
pulp-katello-plugins-0.2-1.el6sat.noarch
ruby193-rubygem-katello-1.5.0-40.el6sat.noarch
rubygem-hammer_cli_katello-0.0.3-22.el6sat.noarch

How reproducible:
100%

Steps to Reproduce:
(1. Change the CDN URL to a local mirror)
2. Visit the Red Hat Repositories page
3. Try to select/enable a repository

Actual results:
Back-end errors

Expected results:
Repo should become enabled

Additional info:
http://projects.theforeman.org/issues/5574

Comment 2 RHEL Program Management 2014-05-19 19:34:51 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 Ivan Necas 2014-05-26 14:57:10 UTC

Created redmine issue http://projects.theforeman.org/issues/5932 from this bug

Comment 5 Ivan Necas 2014-05-27 14:07:13 UTC

Fixed in https://github.com/Katello/katello/pull/4156

Comment 7 Tazim Kolhar 2014-06-02 12:09:45 UTC

VERIFIED

Comment 8 Bryan Kearney 2014-07-02 14:09:48 UTC

This was delivered with 6.0.3, which is the Satellite 6 Beta.

Comment 9 Bryan Kearney 2014-07-02 14:10:00 UTC

This was delivered in 6.0.3, the Beta version of Satellite 6.0

Comment 10 Bryan Kearney 2014-07-02 14:14:41 UTC

This was delivered in the Satellite 6 Beta (6.0.3)