Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add port 443 to listen_ports in apache2 (SOC-9172) #1954

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 7, 2019

  1. Add port 443 to listen_ports in apache2 (SOC-9172)

    This commit stops port clashes between horizon and haproxy on HA
    deployments with SSL enabled.
    
    Without this patch, the apache ssl recipe adds port 443 to the apache
    listen.conf file without adding the port to the listen_ports array.
    In a HA setup with ssl enabled, both HAProxy and Apache will
    try to use port 443. To work around this, the horizon ssl recipe currently
    checks to see what ports are listed in a listen_ports array, and
    if either port 443 or port 80 is in there, it wipes the listen.conf
    file. As port 443 is not currently in the listen_ports array, the
    horizon recipe leaves the listen.conf file alone in cases where
    port 443 is in there.
    
    This commit adds port 443 to the listen_ports array at the point
    where it is added as a listen port, so that the horizon recipe can
    find it later.
    
    More explanation in:
    https://bugzilla.suse.com/show_bug.cgi?id=1141490#c34
    Zarathecat committed Nov 7, 2019
    Configuration menu
    Copy the full SHA
    c5eeab3 View commit details
    Browse the repository at this point in the history