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

Installer and docs disagree on which port the incoming mail SMTP service lives on #3680

Open
jdougan opened this issue Dec 23, 2022 · 5 comments
Labels
documentation Documentation issues

Comments

@jdougan
Copy link
Contributor

jdougan commented Dec 23, 2022

Docs [1] claim it will be installed on port 30025, the last default installs I did it (v303) put it on port 25.

[1] https://docs.sandstorm.io/en/latest/search.html?q=30025,

@ocdtrekkie ocdtrekkie added the documentation Documentation issues label Dec 24, 2022
@ocdtrekkie
Copy link
Collaborator

I'll examine the installer behavior and then update the docs. My guess is this changed when we updated the installer to attempt to use 443 and such for production installs, but since inbound email is barely supported, nobody updated the docs for it.

@zenhack
Copy link
Collaborator

zenhack commented Dec 24, 2022

Yeah, skimming through the install script it looks like there's some logic that tries to use port 25 if it's available.

@jdougan
Copy link
Contributor Author

jdougan commented Jan 23, 2023

Has anything about this been done in some other repo?

@ocdtrekkie
Copy link
Collaborator

Not yet. It's in my inbox still, but I've had a busy few weeks. The installer and the docs are all both in the main repo.

@ocdtrekkie
Copy link
Collaborator

Hmmm, so there are two different things here, and I am unsure the best way to address it in docs:

  1. The Sandstorm installer will set the config file to use port 25 for "full"/"as root" installs if it can.
  2. If SMTP_LISTEN_PORT is not configured in the config, Sandstorm will use port 30025.

If you search for 30025, as above, you'll see comments about what Sandstorm does by default, but it's not what the installer does by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation issues
Projects
None yet
Development

No branches or pull requests

3 participants