From b8fd2b8ec4b14838c57bce78f4353fb1699c23bd Mon Sep 17 00:00:00 2001
From: dannybackx <dannybackx@users.noreply.github.com>
Date: Sat, 22 Apr 2017 13:29:52 +0200
Subject: [PATCH] Reduce debugging overhead. Bugfix in query firewall external
 address.

---
 esp-link/upnp.c | 9 ++++++---
 rest/rest.c     | 6 +++---
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/esp-link/upnp.c b/esp-link/upnp.c
index 30bd00e..67b03bc 100644
--- a/esp-link/upnp.c
+++ b/esp-link/upnp.c
@@ -670,11 +670,13 @@ upnp_tcp_recv_cb(void *arg, char *pdata, unsigned short len) {
     break;
   case upnp_adding_port:
     // FIXME
-    os_printf("UPnP <adding port> TCP Recv len %d, %s\n", len, pdata);
+    // os_printf("UPnP <adding port> TCP Recv len %d, %s\n", len, pdata);
+    os_printf("UPnP <adding port> TCP Recv len %d\n", len);
     break;
   case upnp_removing_port:
     // FIXME
-    os_printf("UPnP <removing port> TCP Recv len %d, %s\n", len, pdata);
+    // os_printf("UPnP <removing port> TCP Recv len %d, %s\n", len, pdata);
+    os_printf("UPnP <removing port> TCP Recv len %d\n", len);
     break;
   case upnp_query_extaddress:
     // <NewExternalIPAddress>83.134.116.129</NewExternalIPAddress>
@@ -682,7 +684,8 @@ upnp_tcp_recv_cb(void *arg, char *pdata, unsigned short len) {
     for (int i=0; i<len; i++) {
       if (strncasecmp(pdata+i, "<NewExternalIPAddress>", 22) == 0) {
 	uint32_t e;
-	if (UTILS_StrToIP(client->host, &e)) {
+	if (UTILS_StrToIP(pdata+i+22, &e)) {
+	  os_printf("Found %08x\n", e);
 	  os_printf("Found %08x\n", e);
 	  client->external_address.addr = e;
 	}
diff --git a/rest/rest.c b/rest/rest.c
index 2100109..2416473 100644
--- a/rest/rest.c
+++ b/rest/rest.c
@@ -316,7 +316,7 @@ REST_SetHeader(CmdPacket *cmd) {
     client->header[len] = '\r';
     client->header[len+1] = '\n';
     client->header[len+2] = 0;
-    DBG_REST("REST: Set header: %s\r\n", client->header);
+    // DBG_REST("REST: Set header: %s\r\n", client->header);
     break;
   case HEADER_CONTENT_TYPE:
     if(client->content_type) os_free(client->content_type);
@@ -325,7 +325,7 @@ REST_SetHeader(CmdPacket *cmd) {
     client->content_type[len] = '\r';
     client->content_type[len+1] = '\n';
     client->content_type[len+2] = 0;
-    DBG_REST("REST: Set content_type: %s\r\n", client->content_type);
+    // DBG_REST("REST: Set content_type: %s\r\n", client->content_type);
     break;
   case HEADER_USER_AGENT:
     if(client->user_agent) os_free(client->user_agent);
@@ -334,7 +334,7 @@ REST_SetHeader(CmdPacket *cmd) {
     client->user_agent[len] = '\r';
     client->user_agent[len+1] = '\n';
     client->user_agent[len+2] = 0;
-    DBG_REST("REST: Set user_agent: %s\r\n", client->user_agent);
+    // DBG_REST("REST: Set user_agent: %s\r\n", client->user_agent);
     break;
   }
 }