add upnp status dictionary to status response

-log upnp debugging information as an error

CHANGELOG.md

@@ -54,6 +54,7 @@ most commands.
   * added `address_unused` command to get existing or generate a new unused address.
   * added pagination support for `address_list`, `channel_list`, `claim_list_mine`,
     `transaction_list` and `utxo_list`.
+  * added `upnp` field to `status` response
   * removed `send_amount_to_address` command previously marked as deprecated
   * removed `channel_list_mine` command previously marked as deprecated
   * removed `get_availability` command previously marked as deprecated

lbrynet/daemon/Components.py

@@ -718,12 +718,18 @@ class UPnPComponent(Component):
                 else:
                     log.info("got external ip from upnp: %s", self.external_ip)
                 yield self._setup_redirects()
-                log.info("set up upnp port redirects")
             except Exception as err:
                 log.warning("error trying to set up upnp: %s", err)
                 self.external_ip = CS.get_external_ip()
         else:
             self.external_ip = CS.get_external_ip()
+        if self.upnp:
+            if not self.upnp_redirects:
+                log.error("failed to setup upnp, debugging infomation: %s", self.upnp.zipped_debugging_info)
+            else:
+                log.info("set up upnp port redirects for gateway: %s", self.upnp.gateway.manufacturer_string)
+        else:
+            log.error("failed to setup upnp")
 
     def stop(self):
         return defer.DeferredList(
@@ -731,6 +737,12 @@ class UPnPComponent(Component):
              for protocol, port in self.upnp_redirects.items()]
         )
 
+    def get_status(self):
+        return {
+            'redirects': self.upnp_redirects,
+            'gateway': '' if not self.upnp else self.upnp.gateway.manufacturer_string,
+        }
+
 
 class ExchangeRateManagerComponent(Component):
     component_name = EXCHANGE_RATE_MANAGER_COMPONENT

lbrynet/daemon/Daemon.py

@@ -771,6 +771,12 @@ class Daemon(AuthJSONRPCServer):
                 },
                 'file_manager': {
                     'managed_files': (int) count of files in the file manager,
+                },
+                'upnp': {
+                    'redirects': {
+                        <TCP | UDP>: (int) external_port,
+                    },
+                    'gateway': (str) manufacturer and model,
                 }
             }
         """