From 6add48e03678b30c6a1adbea4a474b83df5939bd Mon Sep 17 00:00:00 2001 From: Peter Surda Date: Wed, 22 Jun 2016 22:54:06 +0200 Subject: [PATCH] UPnP fix - it looks like some UPnP server implementations consider NewRemoteHost to be a mandatory tag --- src/upnp.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/upnp.py b/src/upnp.py index 324e9663..50b25b8e 100644 --- a/src/upnp.py +++ b/src/upnp.py @@ -110,6 +110,7 @@ class Router: def AddPortMapping(self, externalPort, internalPort, internalClient, protocol, description, leaseDuration = 0, enabled = 1): from debug import logger resp = self.soapRequest(self.upnp_schema + ':1', 'AddPortMapping', [ + ('NewRemoteHost', ''), ('NewExternalPort', str(externalPort)), ('NewProtocol', protocol), ('NewInternalPort', str(internalPort)), @@ -125,6 +126,7 @@ class Router: def DeletePortMapping(self, externalPort, protocol): from debug import logger resp = self.soapRequest(self.upnp_schema + ':1', 'DeletePortMapping', [ + ('NewRemoteHost', ''), ('NewExternalPort', str(externalPort)), ('NewProtocol', protocol), ])