Multiple local IP addresses

[Sergey Ponomarev]

I have a docker installed and it creates it's own network that looks in ifconfig like:

docker0:
        inet 172.17.0.1 netmask 255.255.0.0 broadcast 172.17.255.255
wlp3s0:
        inet 192.168.1.148 netmask 255.255.255.0 broadcast 192.168.1.255

The second interface wlp3s0 is basically my WiFi.

So when I adding a port mapping in the Local IP field it was detected 172.17.0.1 instead of 192.168.1.X
I didn't noticed that and was wondered why the port mapping is not added.

But even when I clicked Advanced and set the local IP explicitly to 192.168.1.X it failed with the error: `606 Auction is not authorized`.
This happened because the local IP sent in the request was set to 0 instead of External port e.g. `<NewInternalPort>0</NewInternalPort>`.

When I set the Local port explicitly it was successful.
BTW this may be a problem on the miniupnpd side because it should be fine with the 0 port and it will just find a first available port e.g. 1025. https://github.com/miniupnp/libnatpmp/issues/29#issuecomment-1411621433
This is anyway not that a user expects so we should make the Local Port same as External Port.

Could you please prefer the 192.168 IP instead of 172.17 IP?
And additionally it would be great to have a dropdown with available IPs.

[Daniele Napolitano (dnax88)]

UPnP Router Control start discovering devices in every interfaces it found.

You can start it with --if <interface> parameter to use only one. Like: upnp-router-control --if wlp3s0

For sure I have to show all the field in the "Add port" window instead of hide local port and IP. Also NewInternalPort as "0" it's a bug.