Bug 1395277

Summary:	handle variables in listing files by ignoring
Product:	Red Hat Satellite	Reporter:	Justin Sherrill <jsherril>
Component:	Repositories	Assignee:	satellite6-bugs <satellite6-bugs>
Status:	CLOSED WONTFIX	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	high
Version:	6.2.0	CC:	bbuckingham, bkearney, jcallaha, jsherril
Target Milestone:	Unspecified	Keywords:	PrioBumpPM, Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-09-04 17:46:32 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 Justin Sherrill 2016-11-15 15:04:08 UTC

Description of problem:

The cdn mistakenly put '$basearch' into a listing file at:

https://cdn.redhat.com/content/dist/rhel/server/7/7.2/listing

And satellite does not handle that as it is trying to do variable substitution. 

It ends up just requesting

https://cdn.redhat.com/content/dist/rhel/server/7/7.2/listing

over and over and over forever.  This is likely due to 

1.  Satellite taking the url https://cdn.redhat.com/content/dist/rhel/server/7/7.2/ and appending $basearch
2.  Performing variable substitution on https://cdn.redhat.com/content/dist/rhel/server/7/7.2/$basearch, resulting in https://cdn.redhat.com/content/dist/rhel/server/7/7.2/
3.  Satellite fetching https://cdn.redhat.com/content/dist/rhel/server/7/7.2/listing again, and repeating at step 1.

Version-Release number of selected component (if applicable):
All versions of satellite

How reproducible:
Always (with a broken CDN)

Steps to Reproduce:
1. Setup a cdn mirror and add '$basearch' to one of the listing files
2. Try to list repos to enable on the redhat repos page that fetches that listing file

Actual results:
Request hangs and list just shows spinner 

Expected results:
repos listed

Additional info:

Comment 2 Brad Buckingham 2016-12-19 19:32:45 UTC

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

Comment 5 Bryan Kearney 2018-09-04 17:46:32 UTC

Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in the product in the foreseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.