-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Support for chaning the HTTP and HTTPS port that hosts will listen on #4127
base: develop
Are you sure you want to change the base?
Conversation
I have seen a few issues that mention that changing the port that nginx listens on is important (especially when using hostNetwork). I am not totally convinced that NPM should allow these use-cases but I was bored enough to implement it anyway. Related Issue: NginxProxyManager#4122
Thank you very much to the developers for their hard work. I sincerely hope to have this feature, as it would allow for more flexible deployment. However, I tested this Docker, and the new feature cannot be used. My docker-compose.yml file is as follows:
|
@csbde could you please give more information what excactly went wrong? My first guess: You are not using hostNeteork and the default ports of 80 and 443 are exposed instead of the defined ports. I could probably improve the documentation in that regard. I will investigate on the weekend, when I have some spare time. |
ReThank you for your reply. I am using the host network mode and can successfully access the management panel through port 81. I can also access the Nginx page via ports 80 and 443, but the defined ports cannot be accessed. Below is the configuration file display I checked after entering the Docker. |
Docker Image for build 3 is available on Note: ensure you backup your NPM instance before testing this image! Especially if there are database changes |
I have seen a few issues that mention that changing the port that nginx listens on is important (especially when using hostNetwork).
I am not totally convinced that NPM should allow these use-cases but I was bored enough to implement it anyway.
Related Issue: #4122