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.

Sign up for GitHub

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

Jump to bottom

Updated README.md to reflect the correct 'DEFAULT_DOMAIN' value. #695

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

Updated README.md to reflect the correct 'DEFAULT_DOMAIN' value. #695

wants to merge 1 commit into from

Commits on Jun 18, 2023

  1. Updated README.md to reflect the correct 'DEFAULT_DOMAIN' value. 

    There is an extremely important bit of information in the Docker .env section that's surprisingly missing.

You see, the most common use case for this app is to be able to host it in a private server in the cloud, front-ended by a reverse proxy. And when we add TLS termination into the MIX as well, it becomes all the more important to advise the general audience that the 'DEFAULT_Domain' is supposed to contain the FQDN value (inclusive of the host portion) and not just the domain portion. If only the domain portion is included, obvious strange behaviours tend to occur, including a scenario where an NGINX reverse proxy redirects any kutt-owned subdomains to a root domain hosted on the same server if the 'DEFAULT_DOMAIN'.
    @john-lazarus
    john-lazarus committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    23c625a View commit details
    Browse the repository at this point in the history