-
Notifications
You must be signed in to change notification settings - Fork 675
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
Changeable Port #2707
Comments
I think this is really expert feature and would cause a lot of trouble to normal users. |
@caco3 In my case, I wanted to port forward both of my devices. My router doesn't have an option to change the change ports, so I had to change it manually.
Can you please explain why it is a expert feature and would cause a lot of trouble? I really missed this feature and I think a lot of users would like to see this. |
@JxxIT It wouldn't have helped you either, since the ESP couldn't connect and you couldn't access the website^^ |
@SybexX Do you mean by changing it directly in the source code? I did that, built, and it worked. |
shit written in the wrong issues^^ |
If it would be a user configurable option, a user, not understanding what he is doing, could change the port number and then lock himself out of the UI as he might not know how to add the port to the URL or forget the exotic number. If you would want it, you could add a PR but it would have to be set in the wlan.ini file so users can not change it by accident. |
I don't know much about port forwarding, but isn't this done by the router/server? |
Normally yes. Eg. the router can route the external port 12345 to the internal port 80 of a specific IP. |
As far as I know, it's switched off on the Fritzbox because it's a security risk or something. |
Yes, obviously it rips a hole into your network. You should only use it when understanding the impact. |
I use two Rpi's with iobroker + Adguard that handle everything and don't allow remote access. |
The Feature
First off, thanks for this amazing tool!
This feature adds a expert setting to change the port off the web interface. Working on a PR.
The text was updated successfully, but these errors were encountered: