Project

Profile

Help

Issue #9089

closed

Pulp Installer Does Not Check ipv6 for Nginx Redirect Rule

Added by jamesmarshall24 over 3 years ago. Updated over 3 years ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
-
Category:
Installer - Moved to GitHub issues
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Master
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

The nginx.conf.j2 template correctly checks ansible facts for available ipv6 and removes those addresses for listening. However, the 301 redirect for port 80 to 443 does not check for available ipv6 addresses and will always write to that configuration.

This is a follow up to #8536

Actions #1

Updated by jamesmarshall24 over 3 years ago

./roles/pulp_webserver/templates/nginx.conf.j2 ln:126-32

{% if not pulp_webserver_disable_https | bool %}
    server {
        listen {{ pulp_webserver_http_port }} default_server;
        listen [::]:{{ pulp_webserver_http_port }} default_server;
        server_name _;
        return 301 https://$host$request_uri;
    }

Added by jamesmarshall24 over 3 years ago

Revision 5e5f0efc | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

Added by jamesmarshall24 over 3 years ago

Revision 5e5f0efc | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

Actions #2

Updated by jamesmarshall24 over 3 years ago

  • Status changed from NEW to MODIFIED

Added by jamesmarshall24 over 3 years ago

Revision c51e9769 | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

(cherry picked from commit 5e5f0efc42fea235ceff79e2c4321cfe7c5b4f00)

Added by jamesmarshall24 over 3 years ago

Revision c51e9769 | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

(cherry picked from commit 5e5f0efc42fea235ceff79e2c4321cfe7c5b4f00)

Actions #4

Updated by mdepaulo@redhat.com over 3 years ago

  • Sprint/Milestone set to 3.14.3

Added by jamesmarshall24 over 3 years ago

Revision e51f6f17 | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

(cherry picked from commit 5e5f0efc42fea235ceff79e2c4321cfe7c5b4f00)

Added by jamesmarshall24 over 3 years ago

Revision e51f6f17 | View on GitHub

Add facts check for ipv6 for nginx server redirect

closes #9089

(cherry picked from commit 5e5f0efc42fea235ceff79e2c4321cfe7c5b4f00)

Also available in: Atom PDF