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

Allow configuration of ports 80 and 443 #142

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mdovey
Copy link

@mdovey mdovey commented Feb 1, 2019

This update allows configuration of the http and https ports as described in issue #134

@TSc2end
Copy link

TSc2end commented May 28, 2019

支持一下,最近被docker下的IPv6+https折磨。。。使用中发现修改docker端口映射的方式修改https端口,在IPv4下是可行的,但是IPv6下却会出现无法访问的情况。而且官方镜像重启后会将NGINX配置文件复原,不利于后续升级,希望能尽早更新此功能

@jmooo
Copy link

jmooo commented Jun 9, 2019

In case anyone else doesn't speak Chinese :)

Support, recently tortured by IPv6+https under docker. . . In the use of modifying the docker port mapping to modify the https port, it is feasible under IPv4, but there will be inaccessibility under IPv6. And the official image will be restored after the NGINX configuration file is restored, which is not conducive to subsequent upgrades. I hope to update this function as soon as possible.

So issue #134 was summarily closed as wontfix, but this sounds like maybe the feature will be added?

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

Successfully merging this pull request may close these issues.

4 participants