Add an option to provide a SITE PORT like you can provide INSTALL PORT in the installation steps

[EkkoKo]

Is your feature request related to a problem? Please describe

I'm currently running in an environment which disables by default running on port 80, yes even if the port is exposed in a container (relates to some internal functionality), I've discovered that the only way to set it up is by changing the config.yaml -> server.http.addr port value.

Describe the solution you'd like

I would love to just have a field that allowed me to set the site port in the installation steps.

Describe alternatives you've considered

The alternative would be editing manually the config.yaml file which isn't that bad but adding it in the installation steps could be nicer.

Notes

I'm more than willing to write a PR to add that functionality :)

[LinkinStars]

@EkkoKo Thank you for your feedback. I do not recommend giving users the option to fill in during the installation process. Instead, I suggest providing an environment variable, similar to the INSTALL_PORT, that can specify the port to run (after installation). This should take precedence over the port specified in the configuration file.

[EkkoKo]

@EkkoKo Thank you for your feedback. I do not recommend giving users the option to fill in during the installation process. Instead, I suggest providing an environment variable, similar to the INSTALL_PORT, that can specify the port to run (after installation). This should take precedence over the port specified in the configuration file.

Yeah I agree, I think it's way better and makes it more dynamic.

[LinkinStars]

I'm more than willing to write a PR to add that functionality :)

👍 Cool, you can try it.

[EkkoKo]

I'm more than willing to write a PR to add that functionality :)

👍 Cool, you can try it.

I'll get started on working on it :)

[EkkoKo]

@LinkinStars I've created a PR and would love some thoughts, (also I'm going to add documenation about the new environment variables in the docs too)

[EkkoKo]

Since the changes were merged, I'm closing this issue.