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

OLMIS-6183, simplify proxy options #73

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

Commits on Apr 25, 2019

  1. OLMIS-6183, simplify proxy options

    Focus is on the nginx config and changing the proxy pass options.  Primarly forwarding the Host header that will include the port as well as the requested address (e.g. nifi.local) and the port.  This allows Nginx to be bound to ports other than 80 and 443.  Further reduced the expected use-cases be assuming that Nginx as a reverse proxy will fully encapsulate Nifi and Superset, and that if a user wants to use SSL, either the termination will be done at Nginx and the configuration will include certs, or the termination may happen a level above Nginx (e.g. an AWS ELB) in which case running a server on port 443 is unneeded.
    joshzamor committed Apr 25, 2019
    Configuration menu
    Copy the full SHA
    6da2686 View commit details
    Browse the repository at this point in the history