Project

General

Profile

Bug #25266

installer should not set pulp's redirect_host to $::fqdn

Added by Evgeni Golov almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Installer
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

If you deploy a Pulp server with "Lazy Sync" enabled, Pulp issues an HTTP redirect when you access a file that has not been downloaded yet to a "streamer". As every redirect, this needs a target host, which can be configured by the redirect_host config option, which is empty by default in Pulp, but our Puppet module sets it to $::fqdn. This means that all redirects will be to the FQDN of the host, and not the name the original HTTP request came in.

The redirection code looks for SERVER_NAME of the request, but uses it only if the redirect_host entry of the config is empty:

https://github.com/pulp/pulp/blob/5b0ce50c49835ddba01b7c2d5744a93c5dba7a9a/server/pulp/server/content/web/views.py#L108-L125

Associated revisions

Revision 31932471 (diff)
Added by Evgeni Golov almost 2 years ago

Fixes #25266 - don't set a default lazy_redirect_host

this allows Pulp to decide itself which host to redirect to

History

#1 Updated by Evgeni Golov almost 2 years ago

  • Pull request https://github.com/theforeman/puppet-pulp/pull/343 added

#2 Updated by Evgeni Golov almost 2 years ago

  • Bugzilla link set to 1639676

#3 Updated by John Mitsch almost 2 years ago

  • Triaged changed from No to Yes
  • Target version set to Katello 3.10.0

#4 Updated by Evgeni Golov almost 2 years ago

  • Status changed from Assigned to Closed

Also available in: Atom PDF