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

readme: SETTING_EXTERNAL_HOST needs to contain a top level domain #413

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

Conversation

karlicoss
Copy link

I am setting up a Zulip instance in my home LAN, so at first tried setting SETTING_EXTERNAL_HOST: "mypchostname:1234".

However this resulted in an Internal Server Error page -- after inspecting the logs, it was due to this line https://github.com/zulip/zulip/blob/81bd63cb46273b8c94ef9e92c00893ed97110119/zerver/models.py#L4904

Seems that FAKE_DOMAIN_NAME isn't exposed for the docker setup, so I decided the easiest option for me was just to add an alias like mypchostname.lan in the hosts file.

But it's a bit of a gotcha -- not sure whether it requires a proper fix, or worth exposing that FAKE_DOMAIN_NAME setting -- so figured at least mention in the docs might help some people.

Without it (e.g. if it's a dotless domain name like `mycomputer`), this results in an Internal Server Error page due to this line https://github.com/zulip/zulip/blob/81bd63cb46273b8c94ef9e92c00893ed97110119/zerver/models.py#L4904
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant