Bug 1099221 - CDN path verification should handle redirects
Summary: CDN path verification should handle redirects
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.3
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: Ivan Necas
QA Contact: Tazim Kolhar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-19 19:15 UTC by Erik M Jacobs
Modified: 2019-09-26 18:15 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-07-02 14:14:41 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 5574 0 None None None 2016-04-22 16:25:51 UTC

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)


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