Merge branch 'jeelabs-master'

Makefile

@@ -164,7 +164,7 @@ EXTRA_INCDIR	=include .
 EXTRA_INCDIR	= include .
 
 # libraries used in this project, mainly provided by the SDK
-LIBS		= c gcc hal phy pp net80211 wpa main lwip
+LIBS		= c gcc hal phy pp net80211 wpa main lwip # crypto ssl
 
 # compiler flags using during compilation of source files
 CFLAGS		+= -Os -ggdb -std=c99 -Werror -Wpointer-arith -Wundef -Wall -Wl,-EL -fno-inline-functions \
@@ -373,8 +373,8 @@ release: all
 	$(Q) egrep -a 'esp-link [a-z0-9.]+ - 201' $(FW_BASE)/user2.bin | cut -b 1-80
 	$(Q) cp $(FW_BASE)/user1.bin $(FW_BASE)/user2.bin $(SDK_BASE)/bin/blank.bin \
 		   "$(SDK_BASE)/bin/boot_v1.4(b1).bin" wiflash release/esp-link-$(BRANCH)
-	$(Q) tar zcf esp-link-$(BRANCH)-$(FLASH_SIZE).tgz -C release esp-link-$(BRANCH)
+	$(Q) tar zcf esp-link-$(BRANCH).tgz -C release esp-link-$(BRANCH)
-	$(Q) echo "Release file: esp-link-$(BRANCH)-$(FLASH_SIZE).tgz"
+	$(Q) echo "Release file: esp-link-$(BRANCH).tgz"
 	$(Q) rm -rf release
 
 clean:

README.md

@@ -26,6 +26,28 @@ Many thanks to https://github.com/brunnels for contributions around the espduino
 For quick support and questions:
 [![Chat at https://gitter.im/jeelabs/esp-link](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/jeelabs/esp-link?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
+Esp-link uses
+-------------
+The simplest use of esp-link is as a transparent serial to wifi bridge. You can flash an attached
+uC over wifi and you can watch the uC's serial debug output by connecting to port 23 or looking
+at the uC Console web page.
+
+The next level is to use the outbound connectivity of esp-link in the uC code. For example, the
+uC can use REST requests to services like thingspeak.com to send sensor values that then get
+stored and plotted by the external service.
+The uC can also use REST requests to retrieve simple configuration
+information or push other forms of notifications. (MQTT functionality is forthcoming.)
+
+An additional option is to add code to esp-link to customize it and put all the communication
+code into esp-link and only keep simple sensor/actuator control in the attached uC. In this
+mode the attached uC sends custom commands to esp-link with sensor/acturator info and
+registers a set of callbacks with esp-link that control sensors/actuators. This way, custom
+commands in esp-link can receive MQTT messages, make simple callbacks into the uC to get sensor
+values or change actuators, and then respond back with MQTT. The way this is architected is that
+the attached uC registers callbacks at start-up such that the code in the esp doesn't need to 
+know which exact sensors/actuators the attached uC has, it learns thta through the initial
+callback registration.
+
 Eye Candy
 ---------
 These screen shots show the Home page, the Wifi configuration page, the console for the
@@ -237,7 +259,22 @@ The attached micro-controller can open outbound TCP connections using a simple
 [serial protocol](https://gist.github.com/tve/a46c44bf1f6b42bc572e).
 More info and sample code forthcoming...
 
+Outbound HTTP REST requests
+---------------------------
+The V2 versions of esp-link support the espduino SLIP protocol that supports simple outbound
+HTTP REST requests. The SLIP protocol consists of commands with binary arguments sent from the
+attached microcontroller to the esp8266, which then performs the command and responds back.
+The responses back use a callback address in the attached microcontroller code, i.e., the
+command sent by the uC contains a callback address and the response from the esp8266 starts
+with that callback address. This enables asynchronous communication where esp-link can notify the
+uC when requests complete or when other actions happen, such as wifi connectivity status changes.
+Support for MQTT is forthcoming.
+
+You can find a demo sketch in a fork of the espduino library at
+https://github.com/tve/espduino in the
+[examples/demo folder](https://github.com/tve/espduino/tree/master/espduino/examples/demo).
+
 Contact
 -------
 If you find problems with esp-link, please create a github issue. If you have a question, please
-use the gitter link at the top of this page.
+use the gitter chat link at the top of this page.

cmd/handlers.c

@@ -94,7 +94,7 @@ cmdCallback* ICACHE_FLASH_ATTR
 CMD_GetCbByName(char* name) {
   char checkname[16];
   os_strncpy(checkname, name, sizeof(checkname));
-  for (uint8_t i = 0; i < sizeof(commands); i++) {
+  for (uint8_t i = 0; i < MAX_CALLBACKS; i++) {
     //os_printf("CMD_GetCbByName: index %d name=%s cb=%p\n", i, callbacks[i].name,
     //  (void *)callbacks[i].callback);
     // if callback doesn't exist or it's null

esp-link/cgi.c

@@ -12,95 +12,112 @@ Some random cgi routines.
  * Heavily modified and enhanced by Thorsten von Eicken in 2015
  * ----------------------------------------------------------------------------
  */
+
+
+#include <esp8266.h>
 #include "cgi.h"
+#include "espfs.h"
+
+void noCacheHeaders(HttpdConnData *connData, int code) {
+  httpdStartResponse(connData, code);
+  httpdHeader(connData, "Cache-Control", "no-cache, no-store, must-revalidate");
+  httpdHeader(connData, "Pragma", "no-cache");
+  httpdHeader(connData, "Expires", "0");
+}
 
 void ICACHE_FLASH_ATTR
 jsonHeader(HttpdConnData *connData, int code) {
-	httpdStartResponse(connData, code);
+  noCacheHeaders(connData, code);
-	httpdHeader(connData, "Cache-Control", "no-cache, no-store, must-revalidate");
+  httpdHeader(connData, "Content-Type", "application/json");
-	httpdHeader(connData, "Pragma", "no-cache");
+  httpdEndHeaders(connData);
-	httpdHeader(connData, "Expires", "0");
-	httpdHeader(connData, "Content-Type", "application/json");
-	httpdEndHeaders(connData);
 }
 
-uint8_t ICACHE_FLASH_ATTR
+void ICACHE_FLASH_ATTR
-UTILS_StrToIP(const char* str, void *ip){
+errorResponse(HttpdConnData *connData, int code, char *message) {
-  /* The count of the number of bytes processed. */
+  noCacheHeaders(connData, code);
-  int i;
+  httpdEndHeaders(connData);
-  /* A pointer to the next digit to process. */
+  httpdSend(connData, message, -1);
-  const char * start;
+  os_printf("HTTP %d error response: \"%s\"\n", code, message);
+}
 
-  start = str;
+// look for the HTTP arg 'name' and store it at 'config' with max length 'max_len' (incl
-  for (i = 0; i < 4; i++) {
+// terminating zero), returns -1 on error, 0 if not found, 1 if found and OK
-    /* The digit being processed. */
+int ICACHE_FLASH_ATTR
-    char c;
+getStringArg(HttpdConnData *connData, char *name, char *config, int max_len) {
-    /* The value of this byte. */
+  char buff[128];
-    int n = 0;
+  int len = httpdFindArg(connData->getArgs, name, buff, sizeof(buff));
-    while (1) {
+  if (len < 0) return 0; // not found, skip
-      c = *start;
+  if (len >= max_len) {
-      start++;
+    os_sprintf(buff, "Value for %s too long (%d > %d allowed)", name, len, max_len-1);
-      if (c >= '0' && c <= '9') {
+    errorResponse(connData, 400, buff);
-        n *= 10;
+    return -1;
-        n += c - '0';
-      }
-      /* We insist on stopping at "." if we are still parsing
-      the first, second, or third numbers. If we have reached
-      the end of the numbers, we will allow any character. */
-      else if ((i < 3 && c == '.') || i == 3) {
-        break;
-      }
-      else {
-        return 0;
-      }
-    }
-    if (n >= 256) {
-      return 0;
-    }
-    ((uint8_t*)ip)[i] = n;
   }
+  strcpy(config, buff);
   return 1;
 }
 
+int ICACHE_FLASH_ATTR
+getBoolArg(HttpdConnData *connData, char *name, bool*config) {
+  char buff[64];
+  int len = httpdFindArg(connData->getArgs, name, buff, sizeof(buff));
+  if (len < 0) return 0; // not found, skip
+
+  if (strcmp(buff, "1") == 0 || strcmp(buff, "true") == 0) {
+    *config = true;
+    return 1;
+  }
+  if (strcmp(buff, "0") == 0 || strcmp(buff, "false") == 0) {
+    *config = false;
+    return 1;
+  }
+  os_sprintf(buff, "Invalid value for %s", name);
+  errorResponse(connData, 400, buff);
+  return -1;
+}
+
+
 #define TOKEN(x) (os_strcmp(token, x) == 0)
 #if 0
 // Handle system information variables and print their value, returns the number of
 // characters appended to buff
 int ICACHE_FLASH_ATTR printGlobalInfo(char *buff, int buflen, char *token) {
-	if (TOKEN("si_chip_id")) {
+  if (TOKEN("si_chip_id")) {
-		return os_sprintf(buff, "0x%x", system_get_chip_id());
+    return os_sprintf(buff, "0x%x", system_get_chip_id());
-	} else if (TOKEN("si_freeheap")) {
+  } else if (TOKEN("si_freeheap")) {
-		return os_sprintf(buff, "%dKB", system_get_free_heap_size()/1024);
+    return os_sprintf(buff, "%dKB", system_get_free_heap_size()/1024);
-	} else if (TOKEN("si_uptime")) {
+  } else if (TOKEN("si_uptime")) {
-		uint32 t = system_get_time() / 1000000; // in seconds
+    uint32 t = system_get_time() / 1000000; // in seconds
-		return os_sprintf(buff, "%dd%dh%dm%ds", t/(24*3600), (t/(3600))%24, (t/60)%60, t%60);
+    return os_sprintf(buff, "%dd%dh%dm%ds", t/(24*3600), (t/(3600))%24, (t/60)%60, t%60);
-	} else if (TOKEN("si_boot_version")) {
+  } else if (TOKEN("si_boot_version")) {
-		return os_sprintf(buff, "%d", system_get_boot_version());
+    return os_sprintf(buff, "%d", system_get_boot_version());
-	} else if (TOKEN("si_boot_address")) {
+  } else if (TOKEN("si_boot_address")) {
-		return os_sprintf(buff, "0x%x", system_get_userbin_addr());
+    return os_sprintf(buff, "0x%x", system_get_userbin_addr());
-	} else if (TOKEN("si_cpu_freq")) {
+  } else if (TOKEN("si_cpu_freq")) {
-		return os_sprintf(buff, "%dMhz", system_get_cpu_freq());
+    return os_sprintf(buff, "%dMhz", system_get_cpu_freq());
-	} else {
+  } else {
-		return 0;
+    return 0;
-	}
+  }
 }
 #endif
 
 extern char *esp_link_version; // in user_main.c
 
 int ICACHE_FLASH_ATTR cgiMenu(HttpdConnData *connData) {
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-	char buff[1024];
+  char buff[1024];
-	// don't use jsonHeader so the response does get cached
+  // don't use jsonHeader so the response does get cached
-	httpdStartResponse(connData, 200);
+  httpdStartResponse(connData, 200);
-	httpdHeader(connData, "Cache-Control", "max-age=3600, must-revalidate");
+  httpdHeader(connData, "Cache-Control", "max-age=3600, must-revalidate");
-	httpdHeader(connData, "Content-Type", "application/json");
+  httpdHeader(connData, "Content-Type", "application/json");
-	httpdEndHeaders(connData);
+  httpdEndHeaders(connData);
-	// construct json response
+  // construct json response
-	os_sprintf(buff,
+  os_sprintf(buff,
-			"{\"menu\": [\"Home\", \"/home.html\", \"Wifi\", \"/wifi/wifi.html\","
+      "{\"menu\": [\"Home\", \"/home.html\", "
-			"\"\xC2\xB5" "C Console\", \"/console.html\", \"Debug log\", \"/log.html\" ],\n"
+      "\"Wifi\", \"/wifi/wifi.html\","
-			" \"version\": \"%s\" }", esp_link_version);
+      "\"\xC2\xB5" "C Console\", \"/console.html\", "
-	httpdSend(connData, buff, -1);
+      "\"REST/MQTT\", \"/mqtt.html\","
-	return HTTPD_CGI_DONE;
+      "\"Debug log\", \"/log.html\" ],\n"
+      " \"version\": \"%s\" }", esp_link_version);
+  httpdSend(connData, buff, -1);
+  return HTTPD_CGI_DONE;
 }

esp-link/cgi.h

@@ -5,6 +5,9 @@
 #include "httpd.h"
 
 void jsonHeader(HttpdConnData *connData, int code);
+void errorResponse(HttpdConnData *connData, int code, char *message);
+int getStringArg(HttpdConnData *connData, char *name, char *config, int max_len);
+int getBoolArg(HttpdConnData *connData, char *name, bool*config);
 int cgiMenu(HttpdConnData *connData);
 uint8_t UTILS_StrToIP(const char* str, void *ip);
 

esp-link/cgimqtt.c

@@ -0,0 +1,111 @@
+// Copyright 2015 by Thorsten von Eicken, see LICENSE.txt
+// // TCP Client settings
+
+#include <esp8266.h>
+#include "cgi.h"
+#include "config.h"
+#include "cgimqtt.h"
+
+// Cgi to return MQTT settings
+int ICACHE_FLASH_ATTR cgiMqttGet(HttpdConnData *connData) {
+  char buff[2048];
+  int len;
+
+  if (connData->conn==NULL) return HTTPD_CGI_DONE;
+
+  len = os_sprintf(buff, "{ "
+      "\"slip-enable\":%d, "
+      "\"mqtt-enable\":%d, "
+      "\"mqtt-status-enable\":%d, "
+      "\"mqtt-port\":%d, "
+      "\"mqtt-host\":\"%s\", "
+      "\"mqtt-client-id\":\"%s\", "
+      "\"mqtt-username\":\"%s\", "
+      "\"mqtt-password\":\"%s\", "
+      "\"mqtt-status-topic\":\"%s\", "
+      "\"mqtt-state\":\"%s\" }",
+      flashConfig.slip_enable, flashConfig.mqtt_enable, flashConfig.mqtt_status_enable,
+      flashConfig.mqtt_port, flashConfig.mqtt_hostname, flashConfig.mqtt_client,
+      flashConfig.mqtt_username, flashConfig.mqtt_password,
+      flashConfig.mqtt_status_topic, "connected");
+
+  jsonHeader(connData, 200);
+  httpdSend(connData, buff, len);
+  return HTTPD_CGI_DONE;
+}
+
+// Cgi to change choice of pin assignments
+int ICACHE_FLASH_ATTR cgiMqttSet(HttpdConnData *connData) {
+  if (connData->conn==NULL) return HTTPD_CGI_DONE;
+
+  // handle MQTT server settings
+  int mqtt_server = 0; // accumulator for changes/errors
+  mqtt_server |= getStringArg(connData, "mqtt-host",
+      flashConfig.mqtt_hostname, sizeof(flashConfig.mqtt_hostname));
+  if (mqtt_server < 0) return HTTPD_CGI_DONE;
+  mqtt_server |= getStringArg(connData, "mqtt-client-id",
+      flashConfig.mqtt_client, sizeof(flashConfig.mqtt_client));
+  if (mqtt_server < 0) return HTTPD_CGI_DONE;
+  mqtt_server |= getStringArg(connData, "mqtt-username",
+      flashConfig.mqtt_username, sizeof(flashConfig.mqtt_username));
+  if (mqtt_server < 0) return HTTPD_CGI_DONE;
+  mqtt_server |= getStringArg(connData, "mqtt-password",
+      flashConfig.mqtt_password, sizeof(flashConfig.mqtt_password));
+  if (mqtt_server < 0) return HTTPD_CGI_DONE;
+  mqtt_server |= getBoolArg(connData, "mqtt-enable",
+      &flashConfig.mqtt_enable);
+
+  // handle mqtt port
+  char buff[16];
+  if (httpdFindArg(connData->getArgs, "mqtt-port", buff, sizeof(buff)) > 0) {
+    int32_t port = atoi(buff);
+    if (port > 0 && port < 65536) {
+      flashConfig.mqtt_port = port;
+      mqtt_server |= 1;
+    } else {
+      errorResponse(connData, 400, "Invalid MQTT port");
+      return HTTPD_CGI_DONE;
+    }
+  }
+
+  // if server setting changed, we need to "make it so"
+  if (mqtt_server) {
+    os_printf("MQTT server settings changed, enable=%d\n", flashConfig.mqtt_enable);
+    // TODO
+  }
+
+  // no action required if mqtt status settings change, they just get picked up at the
+  // next status tick
+  if (getBoolArg(connData, "mqtt-status-enable", &flashConfig.mqtt_status_enable) < 0)
+    return HTTPD_CGI_DONE;
+  if (getStringArg(connData, "mqtt-status-topic",
+        flashConfig.mqtt_status_topic, sizeof(flashConfig.mqtt_status_topic)) < 0)
+    return HTTPD_CGI_DONE;
+
+  // if SLIP-enable is toggled it gets picked-up immediately by the parser
+  int slip_update = getBoolArg(connData, "slip-enable", &flashConfig.slip_enable);
+  if (slip_update < 0) return HTTPD_CGI_DONE;
+  if (slip_update > 0) os_printf("SLIP-enable changed: %d\n", flashConfig.slip_enable);
+
+  os_printf("Saving config\n");
+  if (configSave()) {
+    httpdStartResponse(connData, 200);
+    httpdEndHeaders(connData);
+  } else {
+    httpdStartResponse(connData, 500);
+    httpdEndHeaders(connData);
+    httpdSend(connData, "Failed to save config", -1);
+  }
+  return HTTPD_CGI_DONE;
+}
+
+int ICACHE_FLASH_ATTR cgiMqtt(HttpdConnData *connData) {
+  if (connData->requestType == HTTPD_METHOD_GET) {
+    return cgiMqttGet(connData);
+  } else if (connData->requestType == HTTPD_METHOD_POST) {
+    return cgiMqttSet(connData);
+  } else {
+    jsonHeader(connData, 404);
+    return HTTPD_CGI_DONE;
+  }
+}

esp-link/cgimqtt.h

@@ -0,0 +1,8 @@
+#ifndef CGIMQTT_H
+#define CGIMQTT_H
+
+#include "httpd.h"
+
+int cgiMqtt(HttpdConnData *connData);
+
+#endif

esp-link/cgipins.c

@@ -24,9 +24,9 @@ static const int num_map_func = sizeof(map_func)/sizeof(char*);
 
 // Cgi to return choice of pin assignments
 int ICACHE_FLASH_ATTR cgiPinsGet(HttpdConnData *connData) {
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted
 
-	char buff[2048];
+  char buff[2048];
   int len;
 
   // figure out current mapping
@@ -62,27 +62,27 @@ int ICACHE_FLASH_ATTR cgiPinsGet(HttpdConnData *connData) {
   }
   len += os_sprintf(buff+len, "\n] }");
 
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
-	httpdSend(connData, buff, len);
+  httpdSend(connData, buff, len);
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 // Cgi to change choice of pin assignments
 int ICACHE_FLASH_ATTR cgiPinsSet(HttpdConnData *connData) {
-	if (connData->conn==NULL) {
+  if (connData->conn==NULL) {
-		return HTTPD_CGI_DONE; // Connection aborted
+    return HTTPD_CGI_DONE; // Connection aborted
-	}
+  }
 
   char buff[128];
-	int len = httpdFindArg(connData->getArgs, "map", buff, sizeof(buff));
+  int len = httpdFindArg(connData->getArgs, "map", buff, sizeof(buff));
-	if (len <= 0) {
+  if (len <= 0) {
-	  jsonHeader(connData, 400);
+    jsonHeader(connData, 400);
     return HTTPD_CGI_DONE;
   }
 
   int m = atoi(buff);
-	if (m < 0 || m >= num_map_names) {
+  if (m < 0 || m >= num_map_names) {
-	  jsonHeader(connData, 400);
+    jsonHeader(connData, 400);
     return HTTPD_CGI_DONE;
   }
 
@@ -106,17 +106,17 @@ int ICACHE_FLASH_ATTR cgiPinsSet(HttpdConnData *connData) {
     httpdEndHeaders(connData);
     httpdSend(connData, "Failed to save config", -1);
   }
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 int ICACHE_FLASH_ATTR cgiPins(HttpdConnData *connData) {
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-	if (connData->requestType == HTTPD_METHOD_GET) {
+  if (connData->requestType == HTTPD_METHOD_GET) {
-		return cgiPinsGet(connData);
+    return cgiPinsGet(connData);
-	} else if (connData->requestType == HTTPD_METHOD_POST) {
+  } else if (connData->requestType == HTTPD_METHOD_POST) {
-		return cgiPinsSet(connData);
+    return cgiPinsSet(connData);
-	} else {
+  } else {
-		jsonHeader(connData, 404);
+    jsonHeader(connData, 404);
-		return HTTPD_CGI_DONE;
+    return HTTPD_CGI_DONE;
-	}
+  }
 }

esp-link/cgiwifi.c

@@ -31,12 +31,12 @@ uint8_t wifiState = wifiIsDisconnected;
 // reasons for which a connection failed
 uint8_t wifiReason = 0;
 static char *wifiReasons[] = {
-	"", "unspecified", "auth_expire", "auth_leave", "assoc_expire", "assoc_toomany", "not_authed",
+  "", "unspecified", "auth_expire", "auth_leave", "assoc_expire", "assoc_toomany", "not_authed",
-	"not_assoced", "assoc_leave", "assoc_not_authed", "disassoc_pwrcap_bad", "disassoc_supchan_bad",
+  "not_assoced", "assoc_leave", "assoc_not_authed", "disassoc_pwrcap_bad", "disassoc_supchan_bad",
-	"ie_invalid", "mic_failure", "4way_handshake_timeout", "group_key_update_timeout",
+  "ie_invalid", "mic_failure", "4way_handshake_timeout", "group_key_update_timeout",
-	"ie_in_4way_differs", "group_cipher_invalid", "pairwise_cipher_invalid", "akmp_invalid",
+  "ie_in_4way_differs", "group_cipher_invalid", "pairwise_cipher_invalid", "akmp_invalid",
-	"unsupp_rsn_ie_version", "invalid_rsn_ie_cap", "802_1x_auth_failed", "cipher_suite_rejected",
+  "unsupp_rsn_ie_version", "invalid_rsn_ie_cap", "802_1x_auth_failed", "cipher_suite_rejected",
-	"beacon_timeout", "no_ap_found" };
+  "beacon_timeout", "no_ap_found" };
 
 static char *wifiMode[] = { 0, "STA", "AP", "AP+STA" };
 static char *wifiPhy[]  = { 0, "11b", "11g", "11n" };
@@ -44,51 +44,51 @@ static char *wifiPhy[]  = { 0, "11b", "11g", "11n" };
 void (*wifiStatusCb)(uint8_t); // callback when wifi status changes
 
 static char* ICACHE_FLASH_ATTR wifiGetReason(void) {
-	if (wifiReason <= 24) return wifiReasons[wifiReason];
+  if (wifiReason <= 24) return wifiReasons[wifiReason];
-	if (wifiReason >= 200 && wifiReason <= 201) return wifiReasons[wifiReason-200+24];
+  if (wifiReason >= 200 && wifiReason <= 201) return wifiReasons[wifiReason-200+24];
-	return wifiReasons[1];
+  return wifiReasons[1];
 }
 
 // handler for wifi status change callback coming in from espressif library
 static void ICACHE_FLASH_ATTR wifiHandleEventCb(System_Event_t *evt) {
-	switch (evt->event) {
+  switch (evt->event) {
-	case EVENT_STAMODE_CONNECTED:
+  case EVENT_STAMODE_CONNECTED:
-		wifiState = wifiIsConnected;
+    wifiState = wifiIsConnected;
-		wifiReason = 0;
+    wifiReason = 0;
-		os_printf("Wifi connected to ssid %s, ch %d\n", evt->event_info.connected.ssid,
+    os_printf("Wifi connected to ssid %s, ch %d\n", evt->event_info.connected.ssid,
-				evt->event_info.connected.channel);
+        evt->event_info.connected.channel);
-		statusWifiUpdate(wifiState);
+    statusWifiUpdate(wifiState);
-		break;
+    break;
-	case EVENT_STAMODE_DISCONNECTED:
+  case EVENT_STAMODE_DISCONNECTED:
-		wifiState = wifiIsDisconnected;
+    wifiState = wifiIsDisconnected;
-		wifiReason = evt->event_info.disconnected.reason;
+    wifiReason = evt->event_info.disconnected.reason;
-		os_printf("Wifi disconnected from ssid %s, reason %s (%d)\n",
+    os_printf("Wifi disconnected from ssid %s, reason %s (%d)\n",
-				evt->event_info.disconnected.ssid, wifiGetReason(), evt->event_info.disconnected.reason);
+        evt->event_info.disconnected.ssid, wifiGetReason(), evt->event_info.disconnected.reason);
-		statusWifiUpdate(wifiState);
+    statusWifiUpdate(wifiState);
-		break;
+    break;
-	case EVENT_STAMODE_AUTHMODE_CHANGE:
+  case EVENT_STAMODE_AUTHMODE_CHANGE:
-		os_printf("Wifi auth mode: %d -> %d\n",
+    os_printf("Wifi auth mode: %d -> %d\n",
-				evt->event_info.auth_change.old_mode, evt->event_info.auth_change.new_mode);
+        evt->event_info.auth_change.old_mode, evt->event_info.auth_change.new_mode);
-		break;
+    break;
-	case EVENT_STAMODE_GOT_IP:
+  case EVENT_STAMODE_GOT_IP:
-		wifiState = wifiGotIP;
+    wifiState = wifiGotIP;
-		wifiReason = 0;
+    wifiReason = 0;
-		os_printf("Wifi got ip:" IPSTR ",mask:" IPSTR ",gw:" IPSTR "\n",
+    os_printf("Wifi got ip:" IPSTR ",mask:" IPSTR ",gw:" IPSTR "\n",
-				IP2STR(&evt->event_info.got_ip.ip), IP2STR(&evt->event_info.got_ip.mask),
+        IP2STR(&evt->event_info.got_ip.ip), IP2STR(&evt->event_info.got_ip.mask),
-				IP2STR(&evt->event_info.got_ip.gw));
+        IP2STR(&evt->event_info.got_ip.gw));
-		statusWifiUpdate(wifiState);
+    statusWifiUpdate(wifiState);
-		break;
+    break;
-	case EVENT_SOFTAPMODE_STACONNECTED:
+  case EVENT_SOFTAPMODE_STACONNECTED:
-		os_printf("Wifi AP: station " MACSTR " joined, AID = %d\n",
+    os_printf("Wifi AP: station " MACSTR " joined, AID = %d\n",
-				MAC2STR(evt->event_info.sta_connected.mac), evt->event_info.sta_connected.aid);
+        MAC2STR(evt->event_info.sta_connected.mac), evt->event_info.sta_connected.aid);
-		break;
+    break;
-	case EVENT_SOFTAPMODE_STADISCONNECTED:
+  case EVENT_SOFTAPMODE_STADISCONNECTED:
-		os_printf("Wifi AP: station " MACSTR " left, AID = %d\n",
+    os_printf("Wifi AP: station " MACSTR " left, AID = %d\n",
-				MAC2STR(evt->event_info.sta_disconnected.mac), evt->event_info.sta_disconnected.aid);
+        MAC2STR(evt->event_info.sta_disconnected.mac), evt->event_info.sta_disconnected.aid);
-		break;
+    break;
-	default:
+  default:
-		break;
+    break;
-	}
+  }
 
   for (int i = 0; i < 4; i++) {
     if (wifi_state_change_cb[i] != NULL) (wifi_state_change_cb[i])(wifiState);
@@ -110,16 +110,16 @@ wifiAddStateChangeCb(WifiStateChangeCb cb) {
 
 //WiFi access point data
 typedef struct {
-	char ssid[32];
+  char ssid[32];
-	sint8 rssi;
+  sint8 rssi;
-	char enc;
+  char enc;
 } ApData;
 
 //Scan result
 typedef struct {
-	char scanInProgress; //if 1, don't access the underlying stuff from the webpage.
+  char scanInProgress; //if 1, don't access the underlying stuff from the webpage.
-	ApData **apData;
+  ApData **apData;
-	int noAps;
+  int noAps;
 } ScanResultData;
 
 //Static scan status storage.
@@ -128,109 +128,109 @@ static ScanResultData cgiWifiAps;
 //Callback the code calls when a wlan ap scan is done. Basically stores the result in
 //the cgiWifiAps struct.
 void ICACHE_FLASH_ATTR wifiScanDoneCb(void *arg, STATUS status) {
-	int n;
+  int n;
-	struct bss_info *bss_link = (struct bss_info *)arg;
+  struct bss_info *bss_link = (struct bss_info *)arg;
-
+
-	if (status!=OK) {
+  if (status!=OK) {
-		os_printf("wifiScanDoneCb status=%d\n", status);
+    os_printf("wifiScanDoneCb status=%d\n", status);
-		cgiWifiAps.scanInProgress=0;
+    cgiWifiAps.scanInProgress=0;
-		return;
+    return;
-	}
+  }
-
+
-	//Clear prev ap data if needed.
+  //Clear prev ap data if needed.
-	if (cgiWifiAps.apData!=NULL) {
+  if (cgiWifiAps.apData!=NULL) {
-		for (n=0; n<cgiWifiAps.noAps; n++) os_free(cgiWifiAps.apData[n]);
+    for (n=0; n<cgiWifiAps.noAps; n++) os_free(cgiWifiAps.apData[n]);
-		os_free(cgiWifiAps.apData);
+    os_free(cgiWifiAps.apData);
-	}
+  }
-
+
-	//Count amount of access points found.
+  //Count amount of access points found.
-	n=0;
+  n=0;
-	while (bss_link != NULL) {
+  while (bss_link != NULL) {
-		bss_link = bss_link->next.stqe_next;
+    bss_link = bss_link->next.stqe_next;
-		n++;
+    n++;
-	}
+  }
-	//Allocate memory for access point data
+  //Allocate memory for access point data
-	cgiWifiAps.apData=(ApData **)os_malloc(sizeof(ApData *)*n);
+  cgiWifiAps.apData=(ApData **)os_malloc(sizeof(ApData *)*n);
-	cgiWifiAps.noAps=n;
+  cgiWifiAps.noAps=n;
-	os_printf("Scan done: found %d APs\n", n);
+  os_printf("Scan done: found %d APs\n", n);
-
+
-	//Copy access point data to the static struct
+  //Copy access point data to the static struct
-	n=0;
+  n=0;
-	bss_link = (struct bss_info *)arg;
+  bss_link = (struct bss_info *)arg;
-	while (bss_link != NULL) {
+  while (bss_link != NULL) {
-		if (n>=cgiWifiAps.noAps) {
+    if (n>=cgiWifiAps.noAps) {
-			//This means the bss_link changed under our nose. Shouldn't happen!
+      //This means the bss_link changed under our nose. Shouldn't happen!
-			//Break because otherwise we will write in unallocated memory.
+      //Break because otherwise we will write in unallocated memory.
-			os_printf("Huh? I have more than the allocated %d aps!\n", cgiWifiAps.noAps);
+      os_printf("Huh? I have more than the allocated %d aps!\n", cgiWifiAps.noAps);
-			break;
+      break;
-		}
+    }
-		//Save the ap data.
+    //Save the ap data.
-		cgiWifiAps.apData[n]=(ApData *)os_malloc(sizeof(ApData));
+    cgiWifiAps.apData[n]=(ApData *)os_malloc(sizeof(ApData));
-		cgiWifiAps.apData[n]->rssi=bss_link->rssi;
+    cgiWifiAps.apData[n]->rssi=bss_link->rssi;
-		cgiWifiAps.apData[n]->enc=bss_link->authmode;
+    cgiWifiAps.apData[n]->enc=bss_link->authmode;
-		strncpy(cgiWifiAps.apData[n]->ssid, (char*)bss_link->ssid, 32);
+    strncpy(cgiWifiAps.apData[n]->ssid, (char*)bss_link->ssid, 32);
-		os_printf("bss%d: %s (%d)\n", n+1, (char*)bss_link->ssid, bss_link->rssi);
+    os_printf("bss%d: %s (%d)\n", n+1, (char*)bss_link->ssid, bss_link->rssi);
-
+
-		bss_link = bss_link->next.stqe_next;
+    bss_link = bss_link->next.stqe_next;
-		n++;
+    n++;
-	}
+  }
-	//We're done.
+  //We're done.
-	cgiWifiAps.scanInProgress=0;
+  cgiWifiAps.scanInProgress=0;
 }
 
 static ETSTimer scanTimer;
 static void ICACHE_FLASH_ATTR scanStartCb(void *arg) {
-	os_printf("Starting a scan\n");
+  os_printf("Starting a scan\n");
-	wifi_station_scan(NULL, wifiScanDoneCb);
+  wifi_station_scan(NULL, wifiScanDoneCb);
 }
 
 static int ICACHE_FLASH_ATTR cgiWiFiStartScan(HttpdConnData *connData) {
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
-	if (!cgiWifiAps.scanInProgress) {
+  if (!cgiWifiAps.scanInProgress) {
-		cgiWifiAps.scanInProgress = 1;
+    cgiWifiAps.scanInProgress = 1;
-		os_timer_disarm(&scanTimer);
+    os_timer_disarm(&scanTimer);
-		os_timer_setfn(&scanTimer, scanStartCb, NULL);
+    os_timer_setfn(&scanTimer, scanStartCb, NULL);
-		os_timer_arm(&scanTimer, 1000, 0);
+    os_timer_arm(&scanTimer, 1000, 0);
-	}
+  }
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 static int ICACHE_FLASH_ATTR cgiWiFiGetScan(HttpdConnData *connData) {
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-	char buff[2048];
+  char buff[2048];
-	int len;
+  int len;
-
+
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
-
+
-	if (cgiWifiAps.scanInProgress==1) {
+  if (cgiWifiAps.scanInProgress==1) {
-		//We're still scanning. Tell Javascript code that.
+    //We're still scanning. Tell Javascript code that.
-		len = os_sprintf(buff, "{\n \"result\": { \n\"inProgress\": \"1\"\n }\n}\n");
+    len = os_sprintf(buff, "{\n \"result\": { \n\"inProgress\": \"1\"\n }\n}\n");
-		httpdSend(connData, buff, len);
+    httpdSend(connData, buff, len);
-		return HTTPD_CGI_DONE;
+    return HTTPD_CGI_DONE;
-	}
+  }
-
+
-	len = os_sprintf(buff, "{\"result\": {\"inProgress\": \"0\", \"APs\": [\n");
+  len = os_sprintf(buff, "{\"result\": {\"inProgress\": \"0\", \"APs\": [\n");
-	for (int pos=0; pos<cgiWifiAps.noAps; pos++) {
+  for (int pos=0; pos<cgiWifiAps.noAps; pos++) {
-		len += os_sprintf(buff+len, "{\"essid\": \"%s\", \"rssi\": %d, \"enc\": \"%d\"}%s\n",
+    len += os_sprintf(buff+len, "{\"essid\": \"%s\", \"rssi\": %d, \"enc\": \"%d\"}%s\n",
-				cgiWifiAps.apData[pos]->ssid, cgiWifiAps.apData[pos]->rssi,
+        cgiWifiAps.apData[pos]->ssid, cgiWifiAps.apData[pos]->rssi,
-				cgiWifiAps.apData[pos]->enc, (pos==cgiWifiAps.noAps-1)?"":",");
+        cgiWifiAps.apData[pos]->enc, (pos==cgiWifiAps.noAps-1)?"":",");
-	}
+  }
-	len += os_sprintf(buff+len, "]}}\n");
+  len += os_sprintf(buff+len, "]}}\n");
-	//os_printf("Sending %d bytes: %s\n", len, buff);
+  //os_printf("Sending %d bytes: %s\n", len, buff);
-	httpdSend(connData, buff, len);
+  httpdSend(connData, buff, len);
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 int ICACHE_FLASH_ATTR cgiWiFiScan(HttpdConnData *connData) {
-	if (connData->requestType == HTTPD_METHOD_GET) {
+  if (connData->requestType == HTTPD_METHOD_GET) {
-		return cgiWiFiGetScan(connData);
+    return cgiWiFiGetScan(connData);
-	} else if (connData->requestType == HTTPD_METHOD_POST) {
+  } else if (connData->requestType == HTTPD_METHOD_POST) {
-		return cgiWiFiStartScan(connData);
+    return cgiWiFiStartScan(connData);
-	} else {
+  } else {
-		jsonHeader(connData, 404);
+    jsonHeader(connData, 404);
-		return HTTPD_CGI_DONE;
+    return HTTPD_CGI_DONE;
-	}
+  }
 }
 
 // ===== timers to change state and rescue from failed associations
@@ -243,31 +243,31 @@ static ETSTimer resetTimer;
 // the connect succeeds, this gets the module in STA-only mode. If it fails, it ensures
 // that the module is in STA+AP mode so the user has a chance to recover.
 static void ICACHE_FLASH_ATTR resetTimerCb(void *arg) {
-	int x = wifi_station_get_connect_status();
+  int x = wifi_station_get_connect_status();
-	int m = wifi_get_opmode() & 0x3;
+  int m = wifi_get_opmode() & 0x3;
-	os_printf("Wifi check: mode=%s status=%d\n", wifiMode[m], x);
+  os_printf("Wifi check: mode=%s status=%d\n", wifiMode[m], x);
 
-	if (x == STATION_GOT_IP) {
+  if (x == STATION_GOT_IP) {
-		if (m != 1) {
+    if (m != 1) {
 #ifdef CHANGE_TO_STA
-			// We're happily connected, go to STA mode
+      // We're happily connected, go to STA mode
-			os_printf("Wifi got IP. Going into STA mode..\n");
+      os_printf("Wifi got IP. Going into STA mode..\n");
-			wifi_set_opmode(1);
+      wifi_set_opmode(1);
-			wifi_set_sleep_type(SLEEP_MODE);
+      wifi_set_sleep_type(SLEEP_MODE);
-			os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
+      os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
 #endif
-		}
+    }
-		log_uart(false);
+    log_uart(false);
-		// no more resetTimer at this point, gotta use physical reset to recover if in trouble
+    // no more resetTimer at this point, gotta use physical reset to recover if in trouble
-	} else {
+  } else {
-		if (m != 3) {
+    if (m != 3) {
-			os_printf("Wifi connect failed. Going into STA+AP mode..\n");
+      os_printf("Wifi connect failed. Going into STA+AP mode..\n");
-			wifi_set_opmode(3);
+      wifi_set_opmode(3);
-		}
+    }
-		log_uart(true);
+    log_uart(true);
-		os_printf("Enabling/continuing uart log\n");
+    os_printf("Enabling/continuing uart log\n");
-		os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
+    os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
-	}
+  }
 }
 
 // Temp store for new ap info.
@@ -277,206 +277,206 @@ static ETSTimer reassTimer;
 
 // Callback actually doing reassociation
 static void ICACHE_FLASH_ATTR reassTimerCb(void *arg) {
-	os_printf("Wifi changing association\n");
+  os_printf("Wifi changing association\n");
-	wifi_station_disconnect();
+  wifi_station_disconnect();
-	stconf.bssid_set = 0;
+  stconf.bssid_set = 0;
-	wifi_station_set_config(&stconf);
+  wifi_station_set_config(&stconf);
-	wifi_station_connect();
+  wifi_station_connect();
-	// Schedule check
+  // Schedule check
-	os_timer_disarm(&resetTimer);
+  os_timer_disarm(&resetTimer);
-	os_timer_setfn(&resetTimer, resetTimerCb, NULL);
+  os_timer_setfn(&resetTimer, resetTimerCb, NULL);
-	os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
+  os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
 }
 
 // This cgi uses the routines above to connect to a specific access point with the
 // given ESSID using the given password.
 int ICACHE_FLASH_ATTR cgiWiFiConnect(HttpdConnData *connData) {
-	char essid[128];
+  char essid[128];
-	char passwd[128];
+  char passwd[128];
-
+
-	if (connData->conn==NULL) return HTTPD_CGI_DONE;
+  if (connData->conn==NULL) return HTTPD_CGI_DONE;
-
+
-	int el = httpdFindArg(connData->getArgs, "essid", essid, sizeof(essid));
+  int el = httpdFindArg(connData->getArgs, "essid", essid, sizeof(essid));
-	int pl = httpdFindArg(connData->getArgs, "passwd", passwd, sizeof(passwd));
+  int pl = httpdFindArg(connData->getArgs, "passwd", passwd, sizeof(passwd));
-
+
-	if (el > 0 && pl >= 0) {
+  if (el > 0 && pl >= 0) {
-		//Set to 0 if you want to disable the actual reconnecting bit
+    //Set to 0 if you want to disable the actual reconnecting bit
-		os_strncpy((char*)stconf.ssid, essid, 32);
+    os_strncpy((char*)stconf.ssid, essid, 32);
-		os_strncpy((char*)stconf.password, passwd, 64);
+    os_strncpy((char*)stconf.password, passwd, 64);
-		os_printf("Wifi try to connect to AP %s pw %s\n", essid, passwd);
+    os_printf("Wifi try to connect to AP %s pw %s\n", essid, passwd);
-
+
-		//Schedule disconnect/connect
+    //Schedule disconnect/connect
-		os_timer_disarm(&reassTimer);
+    os_timer_disarm(&reassTimer);
-		os_timer_setfn(&reassTimer, reassTimerCb, NULL);
+    os_timer_setfn(&reassTimer, reassTimerCb, NULL);
-		os_timer_arm(&reassTimer, 1000, 0);
+    os_timer_arm(&reassTimer, 1000, 0);
-		jsonHeader(connData, 200);
+    jsonHeader(connData, 200);
-	} else {
+  } else {
-		jsonHeader(connData, 400);
+    jsonHeader(connData, 400);
-		httpdSend(connData, "Cannot parse ssid or password", -1);
+    httpdSend(connData, "Cannot parse ssid or password", -1);
-	}
+  }
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 static bool parse_ip(char *buff, ip_addr_t *ip_ptr) {
-	char *next = buff; // where to start parsing next integer
+  char *next = buff; // where to start parsing next integer
-	int found = 0;     // number of integers parsed
+  int found = 0;     // number of integers parsed
-	uint32_t ip = 0;   // the ip addres parsed
+  uint32_t ip = 0;   // the ip addres parsed
-	for (int i=0; i<32; i++) { // 32 is just a safety limit
+  for (int i=0; i<32; i++) { // 32 is just a safety limit
-		char c = buff[i];
+    char c = buff[i];
-		if (c == '.' || c == 0) {
+    if (c == '.' || c == 0) {
-			// parse the preceding integer and accumulate into IP address
+      // parse the preceding integer and accumulate into IP address
-			bool last = c == 0;
+      bool last = c == 0;
-			buff[i] = 0;
+      buff[i] = 0;
-			uint32_t v = atoi(next);
+      uint32_t v = atoi(next);
-			ip = ip | ((v&0xff)<<(found*8));
+      ip = ip | ((v&0xff)<<(found*8));
-			next = buff+i+1; // next integer starts after the '.'
+      next = buff+i+1; // next integer starts after the '.'
-			found++;
+      found++;
-			if (last) { // if at end of string we better got 4 integers
+      if (last) { // if at end of string we better got 4 integers
-				ip_ptr->addr = ip;
+        ip_ptr->addr = ip;
-				return found == 4;
+        return found == 4;
-			}
+      }
-			continue;
+      continue;
-		}
+    }
-		if (c < '0' || c > '9') return false;
+    if (c < '0' || c > '9') return false;
-	}
+  }
-	return false;
+  return false;
 }
 
 #define DEBUGIP
 #ifdef DEBUGIP
 static void ICACHE_FLASH_ATTR debugIP() {
-	struct ip_info info;
+  struct ip_info info;
-	if (wifi_get_ip_info(0, &info)) {
+  if (wifi_get_ip_info(0, &info)) {
-		os_printf("\"ip\": \"%d.%d.%d.%d\"\n", IP2STR(&info.ip.addr));
+    os_printf("\"ip\": \"%d.%d.%d.%d\"\n", IP2STR(&info.ip.addr));
-		os_printf("\"netmask\": \"%d.%d.%d.%d\"\n", IP2STR(&info.netmask.addr));
+    os_printf("\"netmask\": \"%d.%d.%d.%d\"\n", IP2STR(&info.netmask.addr));
-		os_printf("\"gateway\": \"%d.%d.%d.%d\"\n", IP2STR(&info.gw.addr));
+    os_printf("\"gateway\": \"%d.%d.%d.%d\"\n", IP2STR(&info.gw.addr));
-		os_printf("\"hostname\": \"%s\"\n", wifi_station_get_hostname());
+    os_printf("\"hostname\": \"%s\"\n", wifi_station_get_hostname());
-	} else {
+  } else {
-		os_printf("\"ip\": \"-none-\"\n");
+    os_printf("\"ip\": \"-none-\"\n");
-	}
+  }
 }
 #endif
 
 // configure Wifi, specifically DHCP vs static IP address based on flash config
 static void ICACHE_FLASH_ATTR configWifiIP() {
-	if (flashConfig.staticip == 0) {
+  if (flashConfig.staticip == 0) {
-		// let's DHCP!
+    // let's DHCP!
-		wifi_station_set_hostname(flashConfig.hostname);
+    wifi_station_set_hostname(flashConfig.hostname);
-		if (wifi_station_dhcpc_status() == DHCP_STARTED)
+    if (wifi_station_dhcpc_status() == DHCP_STARTED)
-			wifi_station_dhcpc_stop();
+      wifi_station_dhcpc_stop();
-		wifi_station_dhcpc_start();
+    wifi_station_dhcpc_start();
-		os_printf("Wifi uses DHCP, hostname=%s\n", flashConfig.hostname);
+    os_printf("Wifi uses DHCP, hostname=%s\n", flashConfig.hostname);
-	} else {
+  } else {
-		// no DHCP, we got static network config!
+    // no DHCP, we got static network config!
-		wifi_station_dhcpc_stop();
+    wifi_station_dhcpc_stop();
-		struct ip_info ipi;
+    struct ip_info ipi;
-		ipi.ip.addr = flashConfig.staticip;
+    ipi.ip.addr = flashConfig.staticip;
-		ipi.netmask.addr = flashConfig.netmask;
+    ipi.netmask.addr = flashConfig.netmask;
-		ipi.gw.addr = flashConfig.gateway;
+    ipi.gw.addr = flashConfig.gateway;
-		wifi_set_ip_info(0, &ipi);
+    wifi_set_ip_info(0, &ipi);
-		os_printf("Wifi uses static IP %d.%d.%d.%d\n", IP2STR(&ipi.ip.addr));
+    os_printf("Wifi uses static IP %d.%d.%d.%d\n", IP2STR(&ipi.ip.addr));
-	}
+  }
 #ifdef DEBUGIP
-	debugIP();
+  debugIP();
 #endif
 }
 
 // Change special settings
 int ICACHE_FLASH_ATTR cgiWiFiSpecial(HttpdConnData *connData) {
-	char dhcp[8];
+  char dhcp[8];
-	char hostname[32];
+  char hostname[32];
-	char staticip[20];
+  char staticip[20];
-	char netmask[20];
+  char netmask[20];
-	char gateway[20];
+  char gateway[20];
-
+
-	if (connData->conn==NULL) return HTTPD_CGI_DONE;
+  if (connData->conn==NULL) return HTTPD_CGI_DONE;
-
+
-	// get args and their string lengths
+  // get args and their string lengths
-	int dl = httpdFindArg(connData->getArgs, "dhcp", dhcp, sizeof(dhcp));
+  int dl = httpdFindArg(connData->getArgs, "dhcp", dhcp, sizeof(dhcp));
-	int hl = httpdFindArg(connData->getArgs, "hostname", hostname, sizeof(hostname));
+  int hl = httpdFindArg(connData->getArgs, "hostname", hostname, sizeof(hostname));
-	int sl = httpdFindArg(connData->getArgs, "staticip", staticip, sizeof(staticip));
+  int sl = httpdFindArg(connData->getArgs, "staticip", staticip, sizeof(staticip));
-	int nl = httpdFindArg(connData->getArgs, "netmask", netmask, sizeof(netmask));
+  int nl = httpdFindArg(connData->getArgs, "netmask", netmask, sizeof(netmask));
-	int gl = httpdFindArg(connData->getArgs, "gateway", gateway, sizeof(gateway));
+  int gl = httpdFindArg(connData->getArgs, "gateway", gateway, sizeof(gateway));
-
+
-	if (!(dl > 0 && hl >= 0 && sl >= 0 && nl >= 0 && gl >= 0)) {
+  if (!(dl > 0 && hl >= 0 && sl >= 0 && nl >= 0 && gl >= 0)) {
-		jsonHeader(connData, 400);
+    jsonHeader(connData, 400);
-		httpdSend(connData, "Request is missing fields", -1);
+    httpdSend(connData, "Request is missing fields", -1);
-		return HTTPD_CGI_DONE;
+    return HTTPD_CGI_DONE;
-	}
+  }
-
+
-	char url[64]; // redirect URL
+  char url[64]; // redirect URL
-	if (os_strcmp(dhcp, "off") == 0) {
+  if (os_strcmp(dhcp, "off") == 0) {
-		// parse static IP params
+    // parse static IP params
-		struct ip_info ipi;
+    struct ip_info ipi;
-		bool ok = parse_ip(staticip, &ipi.ip);
+    bool ok = parse_ip(staticip, &ipi.ip);
-		if (nl > 0) ok = ok && parse_ip(netmask, &ipi.netmask);
+    if (nl > 0) ok = ok && parse_ip(netmask, &ipi.netmask);
-		else IP4_ADDR(&ipi.netmask, 255, 255, 255, 0);
+    else IP4_ADDR(&ipi.netmask, 255, 255, 255, 0);
-		if (gl > 0) ok = ok && parse_ip(gateway, &ipi.gw);
+    if (gl > 0) ok = ok && parse_ip(gateway, &ipi.gw);
-		else ipi.gw.addr = 0;
+    else ipi.gw.addr = 0;
-		if (!ok) {
+    if (!ok) {
-			jsonHeader(connData, 400);
+      jsonHeader(connData, 400);
-			httpdSend(connData, "Cannot parse static IP config", -1);
+      httpdSend(connData, "Cannot parse static IP config", -1);
-			return HTTPD_CGI_DONE;
+      return HTTPD_CGI_DONE;
-		}
+    }
-		// save the params in flash
+    // save the params in flash
-		flashConfig.staticip = ipi.ip.addr;
+    flashConfig.staticip = ipi.ip.addr;
-		flashConfig.netmask = ipi.netmask.addr;
+    flashConfig.netmask = ipi.netmask.addr;
-		flashConfig.gateway = ipi.gw.addr;
+    flashConfig.gateway = ipi.gw.addr;
-		// construct redirect URL
+    // construct redirect URL
-		os_sprintf(url, "{\"url\": \"http://%d.%d.%d.%d\"}", IP2STR(&ipi.ip));
+    os_sprintf(url, "{\"url\": \"http://%d.%d.%d.%d\"}", IP2STR(&ipi.ip));
-
+
-	} else {
+  } else {
-		// no static IP, set hostname
+    // no static IP, set hostname
-		if (hl == 0) os_strcpy(hostname, "esp-link");
+    if (hl == 0) os_strcpy(hostname, "esp-link");
-		flashConfig.staticip = 0;
+    flashConfig.staticip = 0;
-		os_strcpy(flashConfig.hostname, hostname);
+    os_strcpy(flashConfig.hostname, hostname);
-		os_sprintf(url, "{\"url\": \"http://%s\"}", hostname);
+    os_sprintf(url, "{\"url\": \"http://%s\"}", hostname);
-	}
+  }
-
+
-	configSave(); // ignore error...
+  configSave(); // ignore error...
-	// schedule change-over
+  // schedule change-over
-	os_timer_disarm(&reassTimer);
+  os_timer_disarm(&reassTimer);
-	os_timer_setfn(&reassTimer, configWifiIP, NULL);
+  os_timer_setfn(&reassTimer, configWifiIP, NULL);
-	os_timer_arm(&reassTimer, 1000, 0);
+  os_timer_arm(&reassTimer, 1000, 0);
-	// return redirect info
+  // return redirect info
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
-	httpdSend(connData, url, -1);
+  httpdSend(connData, url, -1);
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 //This cgi changes the operating mode: STA / AP / STA+AP
 int ICACHE_FLASH_ATTR cgiWiFiSetMode(HttpdConnData *connData) {
-	int len;
+  int len;
-	char buff[1024];
+  char buff[1024];
-
+
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-
+
-	len=httpdFindArg(connData->getArgs, "mode", buff, sizeof(buff));
+  len=httpdFindArg(connData->getArgs, "mode", buff, sizeof(buff));
-	if (len!=0) {
+  if (len!=0) {
-		int m = atoi(buff);
+    int m = atoi(buff);
-		os_printf("Wifi switching to mode %d\n", m);
+    os_printf("Wifi switching to mode %d\n", m);
-		wifi_set_opmode(m&3);
+    wifi_set_opmode(m&3);
-		if (m == 1) {
+    if (m == 1) {
-			wifi_set_sleep_type(SLEEP_MODE);
+      wifi_set_sleep_type(SLEEP_MODE);
-			// STA-only mode, reset into STA+AP after a timeout
+      // STA-only mode, reset into STA+AP after a timeout
-			os_timer_disarm(&resetTimer);
+      os_timer_disarm(&resetTimer);
-			os_timer_setfn(&resetTimer, resetTimerCb, NULL);
+      os_timer_setfn(&resetTimer, resetTimerCb, NULL);
-			os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
+      os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
-		}
+    }
-		jsonHeader(connData, 200);
+    jsonHeader(connData, 200);
-	} else {
+  } else {
-	  jsonHeader(connData, 400);
+    jsonHeader(connData, 400);
-	}
+  }
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 static char *connStatuses[] = { "idle", "connecting", "wrong password", "AP not found",
                          "failed", "got IP address" };
 
 static char *wifiWarn[] = { 0,
-	"Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(3)\\\">STA+AP mode</a>",
+  "Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(3)\\\">STA+AP mode</a>",
-	"<b>Can't scan in this mode!</b> Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(3)\\\">STA+AP mode</a>",
+  "<b>Can't scan in this mode!</b> Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(3)\\\">STA+AP mode</a>",
-	"Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(1)\\\">STA mode</a>",
+  "Switch to <a href=\\\"#\\\" onclick=\\\"changeWifiMode(1)\\\">STA mode</a>",
 };
 
 #ifdef CHANGE_TO_STA
@@ -487,109 +487,109 @@ static char *wifiWarn[] = { 0,
 
 // print various Wifi information into json buffer
 int ICACHE_FLASH_ATTR printWifiInfo(char *buff) {
-	int len;
+  int len;
-
+
-	struct station_config stconf;
+  struct station_config stconf;
-	wifi_station_get_config(&stconf);
+  wifi_station_get_config(&stconf);
-
+
-	uint8_t op = wifi_get_opmode() & 0x3;
+  uint8_t op = wifi_get_opmode() & 0x3;
-	char *mode = wifiMode[op];
+  char *mode = wifiMode[op];
-	char *status = "unknown";
+  char *status = "unknown";
-	int st = wifi_station_get_connect_status();
+  int st = wifi_station_get_connect_status();
-	if (st >= 0 && st < sizeof(connStatuses)) status = connStatuses[st];
+  if (st >= 0 && st < sizeof(connStatuses)) status = connStatuses[st];
-	int p = wifi_get_phy_mode();
+  int p = wifi_get_phy_mode();
-	char *phy = wifiPhy[p&3];
+  char *phy = wifiPhy[p&3];
-	char *warn = wifiWarn[op];
+  char *warn = wifiWarn[op];
-	sint8 rssi = wifi_station_get_rssi();
+  sint8 rssi = wifi_station_get_rssi();
-	if (rssi > 0) rssi = 0;
+  if (rssi > 0) rssi = 0;
-	uint8 mac_addr[6];
+  uint8 mac_addr[6];
-	wifi_get_macaddr(0, mac_addr);
+  wifi_get_macaddr(0, mac_addr);
-	uint8_t chan = wifi_get_channel();
+  uint8_t chan = wifi_get_channel();
-
+
-	len = os_sprintf(buff,
+  len = os_sprintf(buff,
-		"\"mode\": \"%s\", \"modechange\": \"%s\", \"ssid\": \"%s\", \"status\": \"%s\", \"phy\": \"%s\", "
+    "\"mode\": \"%s\", \"modechange\": \"%s\", \"ssid\": \"%s\", \"status\": \"%s\", \"phy\": \"%s\", "
-		"\"rssi\": \"%ddB\", \"warn\": \"%s\", \"mac\":\"%02x:%02x:%02x:%02x:%02x:%02x\", \"chan\":%d",
+    "\"rssi\": \"%ddB\", \"warn\": \"%s\", \"mac\":\"%02x:%02x:%02x:%02x:%02x:%02x\", \"chan\":%d",
-		mode, MODECHANGE, (char*)stconf.ssid, status, phy, rssi, warn,
+    mode, MODECHANGE, (char*)stconf.ssid, status, phy, rssi, warn,
-		mac_addr[0], mac_addr[1], mac_addr[2], mac_addr[3], mac_addr[4], mac_addr[5], chan);
+    mac_addr[0], mac_addr[1], mac_addr[2], mac_addr[3], mac_addr[4], mac_addr[5], chan);
-
+
-	struct ip_info info;
+  struct ip_info info;
-	if (wifi_get_ip_info(0, &info)) {
+  if (wifi_get_ip_info(0, &info)) {
-		len += os_sprintf(buff+len, ", \"ip\": \"%d.%d.%d.%d\"", IP2STR(&info.ip.addr));
+    len += os_sprintf(buff+len, ", \"ip\": \"%d.%d.%d.%d\"", IP2STR(&info.ip.addr));
-		len += os_sprintf(buff+len, ", \"netmask\": \"%d.%d.%d.%d\"", IP2STR(&info.netmask.addr));
+    len += os_sprintf(buff+len, ", \"netmask\": \"%d.%d.%d.%d\"", IP2STR(&info.netmask.addr));
-		len += os_sprintf(buff+len, ", \"gateway\": \"%d.%d.%d.%d\"", IP2STR(&info.gw.addr));
+    len += os_sprintf(buff+len, ", \"gateway\": \"%d.%d.%d.%d\"", IP2STR(&info.gw.addr));
-		len += os_sprintf(buff+len, ", \"hostname\": \"%s\"", flashConfig.hostname);
+    len += os_sprintf(buff+len, ", \"hostname\": \"%s\"", flashConfig.hostname);
-	} else {
+  } else {
-		len += os_sprintf(buff+len, ", \"ip\": \"-none-\"");
+    len += os_sprintf(buff+len, ", \"ip\": \"-none-\"");
-	}
+  }
-	len += os_sprintf(buff+len, ", \"staticip\": \"%d.%d.%d.%d\"", IP2STR(&flashConfig.staticip));
+  len += os_sprintf(buff+len, ", \"staticip\": \"%d.%d.%d.%d\"", IP2STR(&flashConfig.staticip));
-	len += os_sprintf(buff+len, ", \"dhcp\": \"%s\"", flashConfig.staticip > 0 ? "off" : "on");
+  len += os_sprintf(buff+len, ", \"dhcp\": \"%s\"", flashConfig.staticip > 0 ? "off" : "on");
-
+
-	return len;
+  return len;
 }
 
 int ICACHE_FLASH_ATTR cgiWiFiConnStatus(HttpdConnData *connData) {
-	char buff[1024];
+  char buff[1024];
-	int len;
+  int len;
 
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
 
-	len = os_sprintf(buff, "{");
+  len = os_sprintf(buff, "{");
-	len += printWifiInfo(buff+len);
+  len += printWifiInfo(buff+len);
-	len += os_sprintf(buff+len, ", ");
+  len += os_sprintf(buff+len, ", ");
 
-	if (wifiReason != 0) {
+  if (wifiReason != 0) {
-		len += os_sprintf(buff+len, "\"reason\": \"%s\", ", wifiGetReason());
+    len += os_sprintf(buff+len, "\"reason\": \"%s\", ", wifiGetReason());
-	}
+  }
 
 #if 0
-	// commented out 'cause often the client that requested the change can't get a request in to
+  // commented out 'cause often the client that requested the change can't get a request in to
-	// find out that it succeeded. Better to just wait the std 15 seconds...
+  // find out that it succeeded. Better to just wait the std 15 seconds...
-	int st=wifi_station_get_connect_status();
+  int st=wifi_station_get_connect_status();
-	if (st == STATION_GOT_IP) {
+  if (st == STATION_GOT_IP) {
-		if (wifi_get_opmode() != 1) {
+    if (wifi_get_opmode() != 1) {
-			// Reset into AP-only mode sooner.
+      // Reset into AP-only mode sooner.
-			os_timer_disarm(&resetTimer);
+      os_timer_disarm(&resetTimer);
-			os_timer_setfn(&resetTimer, resetTimerCb, NULL);
+      os_timer_setfn(&resetTimer, resetTimerCb, NULL);
-			os_timer_arm(&resetTimer, 1000, 0);
+      os_timer_arm(&resetTimer, 1000, 0);
-		}
+    }
-	}
+  }
 #endif
 
-	len += os_sprintf(buff+len, "\"x\":0}\n");
+  len += os_sprintf(buff+len, "\"x\":0}\n");
 
-	os_printf("  -> %s\n", buff);
+  os_printf("  -> %s\n", buff);
-	httpdSend(connData, buff, len);
+  httpdSend(connData, buff, len);
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 // Cgi to return various Wifi information
 int ICACHE_FLASH_ATTR cgiWifiInfo(HttpdConnData *connData) {
-	char buff[1024];
+  char buff[1024];
 
-	if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
 
-	os_strcpy(buff, "{");
+  os_strcpy(buff, "{");
-	printWifiInfo(buff+1);
+  printWifiInfo(buff+1);
-	os_strcat(buff, "}");
+  os_strcat(buff, "}");
 
-	jsonHeader(connData, 200);
+  jsonHeader(connData, 200);
-	httpdSend(connData, buff, -1);
+  httpdSend(connData, buff, -1);
-	return HTTPD_CGI_DONE;
+  return HTTPD_CGI_DONE;
 }
 
 // Init the wireless, which consists of setting a timer if we expect to connect to an AP
 // so we can revert to STA+AP mode if we can't connect.
 void ICACHE_FLASH_ATTR wifiInit() {
-	wifi_set_phy_mode(2);
+  wifi_set_phy_mode(2);
-	int x = wifi_get_opmode() & 0x3;
+  int x = wifi_get_opmode() & 0x3;
-	os_printf("Wifi init, mode=%s\n", wifiMode[x]);
+  os_printf("Wifi init, mode=%s\n", wifiMode[x]);
-	configWifiIP();
+  configWifiIP();
-
+
-	wifi_set_event_handler_cb(wifiHandleEventCb);
+  wifi_set_event_handler_cb(wifiHandleEventCb);
-	// check on the wifi in a few seconds to see whether we need to switch mode
+  // check on the wifi in a few seconds to see whether we need to switch mode
-	os_timer_disarm(&resetTimer);
+  os_timer_disarm(&resetTimer);
-	os_timer_setfn(&resetTimer, resetTimerCb, NULL);
+  os_timer_setfn(&resetTimer, resetTimerCb, NULL);
-	os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
+  os_timer_arm(&resetTimer, RESET_TIMEOUT, 0);
 }
 

esp-link/config.c

@@ -5,26 +5,27 @@
 #include <osapi.h>
 #include "config.h"
 #include "espfs.h"
-
+#include "crc16.h"
-// hack: this from LwIP
-extern uint16_t inet_chksum(void *dataptr, uint16_t len);
 
 FlashConfig flashConfig;
 FlashConfig flashDefault = {
   33, 0, 0,
   MCU_RESET_PIN, MCU_ISP_PIN, LED_CONN_PIN, LED_SERIAL_PIN,
   115200,
-  "esp-link\0                       ", // hostname
+  "esp-link\0",                        // hostname
   0, 0x00ffffff, 0,                    // static ip, netmask, gateway
   0,                                   // log mode
   0,                                   // swap uart (don't by default)
   1, 0,                                // tcp_enable, rssi_enable
   "\0",                                // api_key
+  0, 0, 0,                             // slip_enable, mqtt_enable, mqtt_status_enable
+  1833,                                // mqtt port
+  "\0", "\0", "\0", "\0", "\0",        // mqtt host, client, user, password, status-topic
 };
 
 typedef union {
   FlashConfig fc;
-  uint8_t     block[128];
+  uint8_t     block[1024];
 } FlashFull;
 
 // magic number to recognize thet these are our flash settings as opposed to some random stuff
@@ -63,7 +64,7 @@ bool ICACHE_FLASH_ATTR configSave(void) {
   ff.fc.crc = 0;
   //os_printf("cksum of: ");
   //memDump(&ff, sizeof(ff));
-  ff.fc.crc = inet_chksum(&ff, sizeof(ff));
+  ff.fc.crc = crc16_data((unsigned char*)&ff, sizeof(ff), 0);
   //os_printf("cksum is %04x\n", ff.fc.crc);
   // write primary with incorrect seq
   ff.fc.seq = 0xffffffff;
@@ -121,12 +122,16 @@ static int ICACHE_FLASH_ATTR selectPrimary(FlashFull *ff0, FlashFull *ff1) {
   // check CRC of ff0
   uint16_t crc = ff0->fc.crc;
   ff0->fc.crc = 0;
-  bool ff0_crc_ok = inet_chksum(ff0, sizeof(FlashFull)) == crc;
+  bool ff0_crc_ok = crc16_data((unsigned char*)ff0, sizeof(FlashFull), 0) == crc;
+
+  os_printf("FLASH chk=0x%04x crc=0x%04x full_sz=%d sz=%d\n",
+      crc16_data((unsigned char*)ff0, sizeof(FlashFull), 0),
+      crc, sizeof(FlashFull), sizeof(FlashConfig));
 
   // check CRC of ff1
   crc = ff1->fc.crc;
   ff1->fc.crc = 0;
-  bool ff1_crc_ok = inet_chksum(ff1, sizeof(FlashFull)) == crc;
+  bool ff1_crc_ok = crc16_data((unsigned char*)ff1, sizeof(FlashFull), 0) == crc;
 
   // decided which we like better
   if (ff0_crc_ok)

esp-link/config.h

@@ -16,6 +16,11 @@ typedef struct {
   uint8_t  swap_uart;                  // swap uart0 to gpio 13&15
   uint8_t  tcp_enable, rssi_enable;    // TCP client settings
   char     api_key[48];                // RSSI submission API key (Grovestreams for now)
+  uint8_t  slip_enable, mqtt_enable,   // SLIP protocol, MQTT client
+           mqtt_status_enable;         // MQTT status reporting
+  uint16_t mqtt_port;
+  char     mqtt_hostname[32], mqtt_client[48], mqtt_username[32], mqtt_password[32];
+  char     mqtt_status_topic[32];
 } FlashConfig;
 extern FlashConfig flashConfig;
 

esp-link/main.c

@@ -17,6 +17,7 @@
 #include "cgiwifi.h"
 #include "cgipins.h"
 #include "cgitcp.h"
+#include "cgimqtt.h"
 #include "cgiflash.h"
 #include "auth.h"
 #include "espfs.h"
@@ -29,7 +30,7 @@
 #include "log.h"
 #include <gpio.h>
 
-//#define SHOW_HEAP_USE
+#define SHOW_HEAP_USE
 
 //Function that tells the authentication system what users/passwords live on the system.
 //This is disabled in the default build; if you want to try it, enable the authBasic line in
@@ -90,6 +91,7 @@ HttpdBuiltInUrl builtInUrls[] = {
   { "/wifi/special", cgiWiFiSpecial, NULL },
   { "/pins", cgiPins, NULL },
   { "/tcpclient", cgiTcp, NULL },
+  { "/mqtt", cgiMqtt, NULL },
 
   { "*", cgiEspFsHook, NULL }, //Catch-all cgi function for the filesystem
   { NULL, NULL, NULL }
@@ -120,16 +122,22 @@ static char *rst_codes[] = {
 # define VERS_STR(V) VERS_STR_STR(V)
 char* esp_link_version = VERS_STR(VERSION);
 
-//Main routine. Initialize stdout, the I/O, filesystem and the webserver and we're done.
+extern void app_init(void);
+
+// Main routine to initialize esp-link.
 void user_init(void) {
+  uart_init(115200, 115200);
+  logInit(); // must come after init of uart
+  os_delay_us(10000L);
   // get the flash config so we know how to init things
   //configWipe(); // uncomment to reset the config for testing purposes
   bool restoreOk = configRestore();
   // init gpio pin registers
   gpio_init();
+  gpio_output_set(0, 0, 0, (1<<15)); // some people tie it GND, gotta ensure it's disabled
   // init UART
-  uart_init(flashConfig.baud_rate, 115200);
+//  uart_init(flashConfig.baud_rate, 115200);
-  logInit(); // must come after init of uart
+//  logInit(); // must come after init of uart
   // say hello (leave some time to cause break in TX after boot loader's msg
   os_delay_us(10000L);
   os_printf("\n\n** %s\n", esp_link_version);
@@ -163,6 +171,5 @@ void user_init(void) {
 
   os_printf("** esp-link ready\n");
 
-  // call user_main init
+  app_init();
-  init();
 }

esp-link/status.c

@@ -14,56 +14,56 @@ static ETSTimer ledTimer;
 static void ICACHE_FLASH_ATTR setLed(int on) {
   int8_t pin = flashConfig.conn_led_pin;
   if (pin < 0) return; // disabled
-	// LED is active-low
+  // LED is active-low
-	if (on) {
+  if (on) {
-		gpio_output_set(0, (1<<pin), (1<<pin), 0);
+    gpio_output_set(0, (1<<pin), (1<<pin), 0);
-	} else {
+  } else {
-		gpio_output_set((1<<pin), 0, (1<<pin), 0);
+    gpio_output_set((1<<pin), 0, (1<<pin), 0);
-	}
+  }
 }
 
 static uint8_t ledState = 0;
 
 // Timer callback to update the LED
 static void ICACHE_FLASH_ATTR ledTimerCb(void *v) {
-	int time = 1000;
+  int time = 1000;
-
+
-	if (wifiState == wifiGotIP) {
+  if (wifiState == wifiGotIP) {
-		// connected, all is good, solid light with a short dark blip every 3 seconds
+    // connected, all is good, solid light with a short dark blip every 3 seconds
-		ledState = 1-ledState;
+    ledState = 1-ledState;
-		time = ledState ? 2900 : 100;
+    time = ledState ? 2900 : 100;
-	} else if (wifiState == wifiIsConnected) {
+  } else if (wifiState == wifiIsConnected) {
-		// waiting for DHCP, go on/off every second
+    // waiting for DHCP, go on/off every second
-		ledState = 1 - ledState;
+    ledState = 1 - ledState;
-		time = 1000;
+    time = 1000;
-	} else {
+  } else {
-		// not connected
+    // not connected
-		switch (wifi_get_opmode()) {
+    switch (wifi_get_opmode()) {
-		case 1: // STA
+    case 1: // STA
-			ledState = 0;
+      ledState = 0;
-			break;
+      break;
-		case 2: // AP
+    case 2: // AP
-			ledState = 1-ledState;
+      ledState = 1-ledState;
-			time = ledState ? 50 : 1950;
+      time = ledState ? 50 : 1950;
-			break;
+      break;
-		case 3: // STA+AP
+    case 3: // STA+AP
-			ledState = 1-ledState;
+      ledState = 1-ledState;
-			time = ledState ? 50 : 950;
+      time = ledState ? 50 : 950;
-			break;
+      break;
-		}
+    }
-	}
+  }
-
+
-	setLed(ledState);
+  setLed(ledState);
-	os_timer_arm(&ledTimer, time, 0);
+  os_timer_arm(&ledTimer, time, 0);
 }
 
 // change the wifi state indication
 void ICACHE_FLASH_ATTR statusWifiUpdate(uint8_t state) {
-	wifiState = state;
+  wifiState = state;
-	// schedule an update (don't want to run into concurrency issues)
+  // schedule an update (don't want to run into concurrency issues)
-	os_timer_disarm(&ledTimer);
+  os_timer_disarm(&ledTimer);
-	os_timer_setfn(&ledTimer, ledTimerCb, NULL);
+  os_timer_setfn(&ledTimer, ledTimerCb, NULL);
-	os_timer_arm(&ledTimer, 500, 0);
+  os_timer_arm(&ledTimer, 500, 0);
 }
 
 //===== RSSI Status update sent to GroveStreams
@@ -76,59 +76,59 @@ static ETSTimer rssiTimer;
 
 // Timer callback to send an RSSI update to a monitoring system
 static void ICACHE_FLASH_ATTR rssiTimerCb(void *v) {
-	if (!flashConfig.rssi_enable || !flashConfig.tcp_enable || flashConfig.api_key[0]==0)
+  if (!flashConfig.rssi_enable || !flashConfig.tcp_enable || flashConfig.api_key[0]==0)
-		return;
+    return;
-
+
-	sint8 rssi = wifi_station_get_rssi();
+  sint8 rssi = wifi_station_get_rssi();
-	os_printf("timer rssi=%d\n", rssi);
+  os_printf("timer rssi=%d\n", rssi);
-	if (rssi >= 0) return; // not connected or other error
+  if (rssi >= 0) return; // not connected or other error
-
+
-	// compose TCP command
+  // compose TCP command
-	uint8_t chan = MAX_TCP_CHAN-1;
+  uint8_t chan = MAX_TCP_CHAN-1;
-	tcpClientCommand(chan, 'T', "grovestreams.com:80");
+  tcpClientCommand(chan, 'T', "grovestreams.com:80");
-
+
-	// compose http header
+  // compose http header
-	char buf[1024];
+  char buf[1024];
-	int hdrLen = os_sprintf(buf,
+  int hdrLen = os_sprintf(buf,
-			"PUT /api/feed?api_key=%s HTTP/1.0\r\n"
+      "PUT /api/feed?api_key=%s HTTP/1.0\r\n"
-			"Content-Type: application/json\r\n"
+      "Content-Type: application/json\r\n"
-			"Content-Length: XXXXX\r\n\r\n",
+      "Content-Length: XXXXX\r\n\r\n",
-			flashConfig.api_key);
+      flashConfig.api_key);
-
+
-	// http body
+  // http body
-	int dataLen = os_sprintf(buf+hdrLen,
+  int dataLen = os_sprintf(buf+hdrLen,
-			"[{\"compId\":\"%s\", \"streamId\":\"%s\", \"data\":%d}]\r",
+      "[{\"compId\":\"%s\", \"streamId\":\"%s\", \"data\":%d}]\r",
-			flashConfig.hostname, GS_STREAM, rssi);
+      flashConfig.hostname, GS_STREAM, rssi);
-	buf[hdrLen+dataLen++] = 0;
+  buf[hdrLen+dataLen++] = 0;
-	buf[hdrLen+dataLen++] = '\n';
+  buf[hdrLen+dataLen++] = '\n';
-
+
-	// hackish way to fill in the content-length
+  // hackish way to fill in the content-length
-	os_sprintf(buf+hdrLen-9, "%5d", dataLen);
+  os_sprintf(buf+hdrLen-9, "%5d", dataLen);
-	buf[hdrLen-4] = '\r'; // fix-up the \0 inserted by sprintf (hack!)
+  buf[hdrLen-4] = '\r'; // fix-up the \0 inserted by sprintf (hack!)
-
+
-	// send the request off and forget about it...
+  // send the request off and forget about it...
-	for (short i=0; i<hdrLen+dataLen; i++) {
+  for (short i=0; i<hdrLen+dataLen; i++) {
-		tcpClientSendChar(chan, buf[i]);
+    tcpClientSendChar(chan, buf[i]);
-	}
+  }
-	tcpClientSendPush(chan);
+  tcpClientSendPush(chan);
 }
 
 //===== Init status stuff
 
 void ICACHE_FLASH_ATTR statusInit(void) {
-	if (flashConfig.conn_led_pin >= 0) {
+  if (flashConfig.conn_led_pin >= 0) {
-		makeGpio(flashConfig.conn_led_pin);
+    makeGpio(flashConfig.conn_led_pin);
-		setLed(1);
+    setLed(1);
-	}
+  }
-	os_printf("CONN led=%d\n", flashConfig.conn_led_pin);
+  os_printf("CONN led=%d\n", flashConfig.conn_led_pin);
-
+
-	os_timer_disarm(&ledTimer);
+  os_timer_disarm(&ledTimer);
-	os_timer_setfn(&ledTimer, ledTimerCb, NULL);
+  os_timer_setfn(&ledTimer, ledTimerCb, NULL);
-	os_timer_arm(&ledTimer, 2000, 0);
+  os_timer_arm(&ledTimer, 2000, 0);
-
+
-	os_timer_disarm(&rssiTimer);
+  os_timer_disarm(&rssiTimer);
-	os_timer_setfn(&rssiTimer, rssiTimerCb, NULL);
+  os_timer_setfn(&rssiTimer, rssiTimerCb, NULL);
-	os_timer_arm(&rssiTimer, RSSI_INTERVAL, 1); // recurring timer
+  os_timer_arm(&rssiTimer, RSSI_INTERVAL, 1); // recurring timer
 }
 
 

html/console.html

@@ -20,7 +20,7 @@
 <script type="text/javascript">console_url = "/console/text"</script>
 <script src="console.js"></script>
 <script type="text/javascript">
-  var rates = [57600, 115200, 230400, 460800];
+  var rates = [9600, 57600, 115200, 250000];
 
   onLoad(function() {
     fetchText(100, true);

html/mqtt.html

@@ -0,0 +1,99 @@
+  <div id="main">
+    <div class="header">
+      <h1>REST &amp; MQTT</h1>
+    </div>
+
+    <div class="content">
+      <div class="pure-g">
+        <div class="pure-u-1"><div class="card">
+          <p>The REST &amp; MQTT support uses the SLIP protocol over the serial port to enable
+          the attached microcontroller to initiate outbound connections.
+          The REST support lets the uC initiate simple HTTP requests while the MQTT support
+          lets it communicate with an MQTT server bidirectionally at QoS 0 thru 2.</p>
+          <p>The MQTT support is in the form of a built-in client that connects to a server
+          using parameters set below and stored in esp-link's flash settings. This allows 
+          esp-link to take care of connection parameters and disconnect/reconnect operations.</p>
+          <p>The MQTT client also supports sending periodic status messages about esp-link itself,
+          including Wifi RSSI, and free heap memory.</p>
+          <div class="form-horizontal">
+            <input type="checkbox" name="slip-enable"/>
+            <label>Enable SLIP on serial port</label>
+          </div>
+        </div></div>
+      </div>
+      <div class="pure-g">
+        <div class="pure-u-1 pure-u-md-1-2">
+          <div class="card">
+            <h1>MQTT
+              <div id="mqtt-spinner" class="spinner spinner-small"></div>
+            </h1>
+            <form action="#" id="mqtt-form" class="pure-form" hidden>
+              <div class="form-horizontal">
+                <input type="checkbox" name="mqtt-enable"/>
+                <label>Enable MQTT client</label>
+              </div>
+              <div class="form-horizontal">
+                <label>MQTT client state: </label>
+                <b id="mqtt-state"></b>
+              </div>
+              <br>
+              <legend>MQTT server settings</legend>
+              <div class="pure-form-stacked">
+                <label>Server hostname/ip</label>
+                <input type="text" name="mqtt-host"/>
+                <label>Server port/ip</label>
+                <input type="text" name="mqtt-port"/>
+                <label>Client ID</label>
+                <input type="text" name="mqtt-client-id"/>
+                <label>Username</label>
+                <input type="text" name="mqtt-username"/>
+                <label>Password</label>
+                <input type="password" name="mqtt-password"/>
+              </div>
+              <button id="mqtt-button" type="submit" class="pure-button button-primary">
+                Update server settings!
+              </button>
+            </form>
+          </div>
+        </div>
+        <div class="pure-u-1 pure-u-md-1-2">
+          <div class="card">
+            <h1>Status reporting
+              <div id="mqtt-status-spinner" class="spinner spinner-small"></div>
+            </h1>
+            <form action="#" id="mqtt-status-form" class="pure-form" hidden>
+              <div class="form-horizontal">
+                <input type="checkbox" name="mqtt-status-enable"/>
+                <label>Enable status reporting via MQTT</label>
+              </div>
+              <br>
+              <legend>Status reporting settings</legend>
+              <div class="pure-form-stacked">
+                <label>Topic prefix</label>
+                <input type="text" name="mqtt-status-topic"/>
+                Suffixes: rssi, heap-free, ...
+              </div>
+              <button id="mqtt-status-button" type="submit" class="pure-button button-primary">
+                Update status settings!
+              </button>
+            </form>
+          </div>
+          <div class="card">
+            <h1>REST</h1>
+            <p>REST requests are enabled as soon as SLIP is enabled.
+            There are no REST-specific settings.</p>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</div>
+
+<script type="text/javascript">
+onLoad(function() {
+  fetchMqtt();
+  bnd($("#mqtt-form"), "submit", changeMqtt);
+  bnd($("#mqtt-status-form"), "submit", changeMqttStatus);
+});
+</script>
+</body></html>

html/style.css

@@ -3,6 +3,10 @@ html, button, input, select, textarea, .pure-g [class *= "pure-u"] {
 	font-family: sans-serif;
 }
 
+input[type="text"], input[type="password"] {
+	width: 100%;
+}
+
 body {
     color: #777;
 }

html/ui.js

@@ -316,44 +316,44 @@ function createInputForPin(pin) {
   input.type = "radio";
   input.name = "pins";
   input.data = pin.name;
-	input.className = "pin-input";
+  input.className = "pin-input";
   input.value= pin.value;
   input.id   = "opt-" + pin.value;
   if (currPin == pin.name) input.checked = "1";
 
-	var descr = m('<label for="opt-'+pin.value+'"><b>'+pin.name+":</b>"+pin.descr+"</label>");
+  var descr = m('<label for="opt-'+pin.value+'"><b>'+pin.name+":</b>"+pin.descr+"</label>");
-	var div = document.createElement("div");
+  var div = document.createElement("div");
-	div.appendChild(input);
+  div.appendChild(input);
-	div.appendChild(descr);
+  div.appendChild(descr);
-	return div;
+  return div;
 }
 
 function displayPins(resp) {
-	var po = $("#pin-mux");
+  var po = $("#pin-mux");
-	po.innerHTML = "";
+  po.innerHTML = "";
-	currPin = resp.curr;
+  currPin = resp.curr;
-	resp.map.forEach(function(v) {
+  resp.map.forEach(function(v) {
-		po.appendChild(createInputForPin(v));
+    po.appendChild(createInputForPin(v));
-	});
+  });
-	var i, inputs = $(".pin-input");
+  var i, inputs = $(".pin-input");
-	for (i=0; i<inputs.length; i++) {
+  for (i=0; i<inputs.length; i++) {
-		inputs[i].onclick = function() { setPins(this.value, this.data) };
+    inputs[i].onclick = function() { setPins(this.value, this.data) };
-	};
+  };
 }
 
 function fetchPins() {
   ajaxJson("GET", "/pins", displayPins, function() {
-		window.setTimeout(fetchPins, 1000);
+    window.setTimeout(fetchPins, 1000);
-	});
+  });
 }
 
 function setPins(v, name) {
   ajaxSpin("POST", "/pins?map="+v, function() {
-		showNotification("Pin assignment changed to " + name);
+    showNotification("Pin assignment changed to " + name);
-	}, function() {
+  }, function() {
-		showNotification("Pin assignment change failed");
+    showNotification("Pin assignment change failed");
-		window.setTimeout(fetchPins, 100);
+    window.setTimeout(fetchPins, 100);
-	});
+  });
 }
 
 //===== TCP client card
@@ -374,11 +374,11 @@ function changeTcpClient(e) {
   addClass(cb, 'pure-button-disabled');
   ajaxSpin("POST", url, function(resp) {
       removeClass(cb, 'pure-button-disabled');
-      getWifiInfo();
+      fetchTcpClient();
     }, function(s, st) {
       showWarning("Error: "+st);
       removeClass(cb, 'pure-button-disabled');
-      getWifiInfo();
+      fetchTcpClient();
     });
 }
 
@@ -390,8 +390,84 @@ function displayTcpClient(resp) {
 
 function fetchTcpClient() {
   ajaxJson("GET", "/tcpclient", displayTcpClient, function() {
-		window.setTimeout(fetchTcpClient, 1000);
+    window.setTimeout(fetchTcpClient, 1000);
-	});
+  });
 }
 
+//===== MQTT cards
+
+function changeMqtt(e) {
+  e.preventDefault();
+  var url = "mqtt?1=1";
+  var i, inputs = document.querySelectorAll('#mqtt-form input');
+  for (i=0; i<inputs.length; i++) {
+    if (inputs[i].type != "checkbox")
+      url += "&" + inputs[i].name + "=" + inputs[i].value;
+  };
+
+  hideWarning();
+  var cb = $("#mqtt-button");
+  addClass(cb, 'pure-button-disabled');
+  ajaxSpin("POST", url, function(resp) {
+      showNotification("MQTT updated");
+      removeClass(cb, 'pure-button-disabled');
+    }, function(s, st) {
+      showWarning("Error: "+st);
+      removeClass(cb, 'pure-button-disabled');
+      window.setTimeout(fetchMqtt, 100);
+    });
+}
+
+function displayMqtt(data) {
+  Object.keys(data).forEach(function(v) {
+    el = $("#" + v);
+    if (el != null) {
+      if (el.nodeName === "INPUT") el.value = data[v];
+      else el.innerHTML = data[v];
+      return;
+    }
+    el = document.querySelector('input[name="' + v + '"]');
+    if (el != null) {
+      if (el.type == "checkbox") el.checked = data[v] > 0;
+      else el.value = data[v];
+    }
+  });
+  $("#mqtt-spinner").setAttribute("hidden", "");
+  $("#mqtt-status-spinner").setAttribute("hidden", "");
+  $("#mqtt-form").removeAttribute("hidden");
+  $("#mqtt-status-form").removeAttribute("hidden");
+
+  var i, inputs = $("input");
+  for (i=0; i<inputs.length; i++) {
+    if (inputs[i].type == "checkbox")
+      inputs[i].onclick = function() { console.log(this); setMqtt(this.name, this.checked) };
+  }
+}
+
+function fetchMqtt() {
+  ajaxJson("GET", "/mqtt", displayMqtt, function() {
+    window.setTimeout(fetchMqtt, 1000);
+  });
+}
+
+function changeMqttStatus(e) {
+  e.preventDefault();
+  var v = document.querySelector('input[name="mqtt-status-topic"]').value;
+  ajaxSpin("POST", "/mqtt?mqtt-status-topic=" + v, function() {
+    showNotification("MQTT status settings updated");
+    }, function(s, st) {
+      showWarning("Error: "+st);
+      window.setTimeout(fetchMqtt, 100);
+    });
+}
+
+function setMqtt(name, v) {
+  ajaxSpin("POST", "/mqtt?" + name + "=" + (v ? 1 : 0), function() {
+    var n = name.replace("-enable", "");
+    showNotification(n + " is now " + (v ? "enabled" : "disabled"));
+    }, function() {
+      showWarning("Enable/disable failed");
+      window.setTimeout(fetchMqtt, 100);
+    });
+}
 

httpd/httpd.c

@@ -191,7 +191,7 @@ int ICACHE_FLASH_ATTR httpdFindArg(char *line, char *arg, char *buff, int buffLe
 		p=(char*)os_strstr(p, "&");
 		if (p!=NULL) p+=1;
 	}
-	os_printf("Finding %s in %s: Not found :/\n", arg, line);
+	//os_printf("Finding %s in %s: Not found :/\n", arg, line);
 	return -1; //not found
 }
 
@@ -521,7 +521,7 @@ static void ICACHE_FLASH_ATTR httpdDisconCb(void *arg) {
 }
 
 // Callback indicating a failure in the connection. "Recon" is probably intended in the sense
-// of "you need to reconnect". Sigh... Note that there is no DiconCb after ReconCb
+// of "you need to reconnect". Sigh... Note that there is no DisconCb after ReconCb
 static void ICACHE_FLASH_ATTR httpdReconCb(void *arg, sint8 err) {
 	debugConn(arg, "httpdReconCb");
 	HttpdConnData *conn = httpdFindConnData(arg);

mqtt/mqtt.h

@@ -27,117 +27,101 @@
 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 * POSSIBILITY OF SUCH DAMAGE.
 */
-#ifndef USER_AT_MQTT_H_
+#ifndef MQTT_H_
-#define USER_AT_MQTT_H_
+#define MQTT_H_
 
-#include <esp8266.h>
 #include "mqtt_msg.h"
-#include "queue.h"
+#include "pktbuf.h"
 
-typedef struct mqtt_event_data_t {
+// in rest.c
-  uint8_t type;
+uint8_t UTILS_StrToIP(const char* str, void *ip);
-  const char* topic;
-  const char* data;
-  uint16_t topic_length;
-  uint16_t data_length;
-  uint16_t data_offset;
-} mqtt_event_data_t;
-
-typedef struct mqtt_state_t {
-  uint16_t port;
-  int auto_reconnect;
-  mqtt_connect_info_t* connect_info;
-  uint8_t* in_buffer;
-  uint8_t* out_buffer;
-  int in_buffer_length;
-  int out_buffer_length;
-  uint16_t message_length;
-  uint16_t message_length_read;
-  mqtt_message_t* outbound_message;
-  mqtt_connection_t mqtt_connection;
-  uint16_t pending_msg_id;
-  int pending_msg_type;
-  int pending_publish_qos;
-} mqtt_state_t;
 
+// State of MQTT connection
 typedef enum {
-  WIFI_INIT,
+  MQTT_DISCONNECTED,    // we're in disconnected state
-  WIFI_CONNECTING,
+  TCP_RECONNECT_REQ,    // connect failed, needs reconnecting
-  WIFI_CONNECTING_ERROR,
+  TCP_CONNECTING,       // in TCP connection process
-  WIFI_CONNECTED,
+  MQTT_CONNECTED,       // conneted (or connecting)
-  DNS_RESOLVE,
-  TCP_DISCONNECTED,
-  TCP_RECONNECT_REQ,
-  TCP_RECONNECT,
-  TCP_CONNECTING,
-  TCP_CONNECTING_ERROR,
-  TCP_CONNECTED,
-  MQTT_CONNECT_SEND,
-  MQTT_CONNECT_SENDING,
-  MQTT_SUBSCIBE_SEND,
-  MQTT_SUBSCIBE_SENDING,
-  MQTT_DATA,
-  MQTT_PUBLISH_RECV,
-  MQTT_PUBLISHING
 } tConnState;
 
+// Simple notification callback
 typedef void (*MqttCallback)(uint32_t* args);
-typedef void (*MqttDataCallback)(uint32_t* args, const char* topic, uint32_t topic_len, const char* data, uint32_t lengh);
+// Callback with data messge
+typedef void (*MqttDataCallback)(uint32_t* args, const char* topic, uint32_t topic_len,
+    const char* data, uint32_t data_len);
 
+// MQTTY client data structure
 typedef struct {
-  struct espconn* pCon;
+  struct espconn*     pCon;                   // socket
-  uint8_t security;
+  // connection information
-  char* host;
+  char*               host;                   // MQTT server
-  uint32_t port;
+  uint16_t            port;
-  ip_addr_t ip;
+  uint8_t             security;               // 0=tcp, 1=ssl
-  mqtt_state_t mqtt_state;
+  ip_addr_t           ip;                     // MQTT server IP address
-  mqtt_connect_info_t connect_info;
+  mqtt_connect_info_t connect_info;           // info to connect/reconnect
-  MqttCallback connectedCb;
+  // protocol state and message assembly
-  MqttCallback cmdConnectedCb;
+  tConnState          connState;              // connection state
-  MqttCallback disconnectedCb;
+  bool                sending;                // espconn_send is pending
-  MqttCallback cmdDisconnectedCb;
+  mqtt_connection_t   mqtt_connection;        // message assembly descriptor
-  MqttCallback tcpDisconnectedCb;
+  PktBuf*             msgQueue;               // queued outbound messages
-  MqttCallback cmdTcpDisconnectedCb;
+  // TCP input buffer
-  MqttCallback publishedCb;
+  uint8_t*            in_buffer;
-  MqttCallback cmdPublishedCb;
+  int                 in_buffer_size;         // length allocated
-  MqttDataCallback dataCb;
+  int                 in_buffer_filled;       // number of bytes held
-  MqttDataCallback cmdDataCb;
+  // outstanding message when we expect an ACK
-  ETSTimer mqttTimer;
+  PktBuf*             pending_buffer;         // buffer sent and awaiting ACK
-  uint32_t keepAliveTick;
+  PktBuf*             sending_buffer;         // buffer sent not awaiting ACK
-  uint32_t reconnectTick;
+  // timer and associated timeout counters
-  uint32_t sendTimeout;
+  ETSTimer            mqttTimer;              // timer for this connection
-  tConnState connState;
+  uint8_t             keepAliveTick;          // seconds 'til keep-alive is required (0=no k-a)
-  QUEUE msgQueue;
+  uint8_t             keepAliveAckTick;       // seconds 'til keep-alive ack is overdue (0=no k-a)
-  void* user_data;
+  uint8_t             timeoutTick;            // seconds 'til other timeout
+  uint8_t             sendTimeout;            // value of send timeout setting
+  // callbacks
+  MqttCallback        connectedCb;
+  MqttCallback        cmdConnectedCb;
+  MqttCallback        disconnectedCb;
+  MqttCallback        cmdDisconnectedCb;
+  MqttCallback        tcpDisconnectedCb;
+  MqttCallback        cmdTcpDisconnectedCb;
+  MqttCallback        publishedCb;
+  MqttCallback        cmdPublishedCb;
+  MqttDataCallback    dataCb;
+  MqttDataCallback    cmdDataCb;
+  // misc
+  void*               user_data;
 } MQTT_Client;
 
-#define SEC_NONSSL 0
+// Initialize client data structure
-#define SEC_SSL	1
+void MQTT_Init(MQTT_Client* mqttClient, char* host, uint32 port,
+    uint8_t security, uint8_t sendTimeout,
+    char* client_id, char* client_user, char* client_pass,
+    uint8_t keepAliveTime, uint8_t cleanSession);
+
+// Set Last Will Topic on client, must be called before MQTT_InitConnection
+void MQTT_InitLWT(MQTT_Client* mqttClient, char* will_topic, char* will_msg,
+    uint8_t will_qos, uint8_t will_retain);
+
+// Kick of a persistent connection to the broker, will reconnect anytime conn breaks
+void MQTT_Connect(MQTT_Client* mqttClient);
+
+// Kill persistent connection
+void MQTT_Disconnect(MQTT_Client* mqttClient);
 
-#define MQTT_FLAG_CONNECTED 	1
+// Subscribe to a topic
-#define MQTT_FLAG_READY 		2
+bool MQTT_Subscribe(MQTT_Client* client, char* topic, uint8_t qos);
-#define MQTT_FLAG_EXIT 			4
 
-#define MQTT_EVENT_TYPE_NONE 			0
+// Publish a message
-#define MQTT_EVENT_TYPE_CONNECTED 		1
+bool MQTT_Publish(MQTT_Client* client, const char* topic, const char* data,
-#define MQTT_EVENT_TYPE_DISCONNECTED 	2
+    uint8_t qos, uint8_t retain);
-#define MQTT_EVENT_TYPE_SUBSCRIBED 		3
-#define MQTT_EVENT_TYPE_UNSUBSCRIBED 	4
-#define MQTT_EVENT_TYPE_PUBLISH 		5
-#define MQTT_EVENT_TYPE_PUBLISHED 		6
-#define MQTT_EVENT_TYPE_EXITED 			7
-#define MQTT_EVENT_TYPE_PUBLISH_CONTINUATION 8
 
-void MQTT_InitConnection(MQTT_Client* mqttClient, char* host, uint32 port, uint8_t security);
+// Callback when connected
-void MQTT_InitClient(MQTT_Client* mqttClient, char* client_id, char* client_user, char* client_pass, uint8_t keepAliveTime, uint8_t cleanSession);
-void MQTT_InitLWT(MQTT_Client* mqttClient, char* will_topic, char* will_msg, uint8_t will_qos, uint8_t will_retain);
 void MQTT_OnConnected(MQTT_Client* mqttClient, MqttCallback connectedCb);
+// Callback when disconnected
 void MQTT_OnDisconnected(MQTT_Client* mqttClient, MqttCallback disconnectedCb);
+// Callback when publish succeeded
 void MQTT_OnPublished(MQTT_Client* mqttClient, MqttCallback publishedCb);
+// Callback when data arrives for subscription
 void MQTT_OnData(MQTT_Client* mqttClient, MqttDataCallback dataCb);
-bool MQTT_Subscribe(MQTT_Client* client, char* topic, uint8_t qos);
-void MQTT_Connect(MQTT_Client* mqttClient);
-void MQTT_Disconnect(MQTT_Client* mqttClient);
-bool MQTT_Publish(MQTT_Client* client, const char* topic, const char* data, uint8_t qos, uint8_t retain);
 
 #endif /* USER_AT_MQTT_H_ */

mqtt/mqtt_cmd.c

@@ -1,3 +1,5 @@
+#include <esp8266.h>
+#include "mqtt.h"
 #include "mqtt_cmd.h"
 
 uint32_t connectedCb = 0, disconnectCb = 0, tcpDisconnectedCb = 0, publishedCb = 0, dataCb = 0;
@@ -65,10 +67,12 @@ MQTTCMD_Setup(CmdPacket *cmd) {
   // create mqtt client
   uint8_t clientLen = sizeof(MQTT_Client);
   MQTT_Client* client = (MQTT_Client*)os_zalloc(clientLen);
-  if (client == NULL)
+  if (client == NULL) return 0;
-    return 0;
   os_memset(client, 0, clientLen);
 
+  return 0;
+#if 0
+
   uint16_t len;
   uint8_t *client_id, *user_data, *pass_data;
   uint32_t keepalive, clean_session, cb_data;
@@ -136,6 +140,7 @@ MQTTCMD_Setup(CmdPacket *cmd) {
   os_free(pass_data);
 
   return (uint32_t)client;
+#endif
 }
 
 uint32_t ICACHE_FLASH_ATTR
@@ -217,7 +222,7 @@ MQTTCMD_Connect(CmdPacket *cmd) {
   // get security
   CMD_PopArg(&req, (uint8_t*)&client->security, 4);
 
-  os_printf("MQTT: MQTTCMD_Connect host=%s, port=%ld, security=%d\n",
+  os_printf("MQTT: MQTTCMD_Connect host=%s, port=%d, security=%d\n",
     client->host,
     client->port,
     client->security);

mqtt/mqtt_cmd.h

@@ -2,7 +2,6 @@
 #define MODULES_MQTT_CMD_H_
 
 #include "cmd.h"
-#include "mqtt.h"
 
 typedef struct {
   uint32_t connectedCb;

mqtt/mqtt_msg.c

@@ -29,6 +29,7 @@
 *
 */
 
+#include <esp8266.h>
 #include "mqtt_msg.h"
 #define MQTT_MAX_FIXED_HEADER_SIZE 3
 
@@ -129,7 +130,7 @@ mqtt_msg_init(mqtt_connection_t* connection, uint8_t* buffer, uint16_t buffer_le
 }
 
 int ICACHE_FLASH_ATTR
-mqtt_get_total_length(uint8_t* buffer, uint16_t length) {
+mqtt_get_total_length(const uint8_t* buffer, uint16_t length) {
   int i;
   int totlen = 0;
 
@@ -146,7 +147,7 @@ mqtt_get_total_length(uint8_t* buffer, uint16_t length) {
 }
 
 const char* ICACHE_FLASH_ATTR
-mqtt_get_publish_topic(uint8_t* buffer, uint16_t* length) {
+mqtt_get_publish_topic(const uint8_t* buffer, uint16_t* length) {
   int i;
   int totlen = 0;
   int topiclen;
@@ -173,7 +174,7 @@ mqtt_get_publish_topic(uint8_t* buffer, uint16_t* length) {
 }
 
 const char* ICACHE_FLASH_ATTR
-mqtt_get_publish_data(uint8_t* buffer, uint16_t* length) {
+mqtt_get_publish_data(const uint8_t* buffer, uint16_t* length) {
   int i;
   int totlen = 0;
   int topiclen;
@@ -215,7 +216,7 @@ mqtt_get_publish_data(uint8_t* buffer, uint16_t* length) {
 }
 
 uint16_t ICACHE_FLASH_ATTR
-mqtt_get_id(uint8_t* buffer, uint16_t length) {
+mqtt_get_id(const uint8_t* buffer, uint16_t length) {
   if (length < 1)
     return 0;
 

mqtt/mqtt_msg.h

@@ -30,8 +30,9 @@
 */
 
 #ifndef MQTT_MSG_H
-#define	MQTT_MSG_H
+#define MQTT_MSG_H
-#include <esp8266.h>
+
+#define PROTOCOL_NAMEv311
 
 enum mqtt_message_type {
   MQTT_MSG_TYPE_CONNECT = 1,
@@ -50,21 +51,22 @@ enum mqtt_message_type {
   MQTT_MSG_TYPE_DISCONNECT = 14
 };
 
+// Descriptor for a serialized MQTT message, this is returned by functions that compose a message
+// (It's really an MQTT packet in v3.1.1 terminology)
 typedef struct mqtt_message {
   uint8_t* data;
   uint16_t length;
-
 } mqtt_message_t;
 
+// Descriptor for a connection with message assembly storage
 typedef struct mqtt_connection {
-  mqtt_message_t message;
+  mqtt_message_t message; // resulting message
-
+  uint16_t message_id;	  // id of assembled message and memo to calculate next message id
-  uint16_t message_id;
+  uint8_t* buffer;	  // buffer for assembling messages
-  uint8_t* buffer;
+  uint16_t buffer_length; // buffer length
-  uint16_t buffer_length;
-
 } mqtt_connection_t;
 
+// Descriptor for a connect request
 typedef struct mqtt_connect_info {
   char* client_id;
   char* username;
@@ -75,32 +77,40 @@ typedef struct mqtt_connect_info {
   uint8_t will_qos;
   uint8_t will_retain;
   uint8_t clean_session;
-
 } mqtt_connect_info_t;
 
-
+static inline int ICACHE_FLASH_ATTR mqtt_get_type(const uint8_t* buffer) {
-static inline int ICACHE_FLASH_ATTR mqtt_get_type(uint8_t* buffer) {
   return (buffer[0] & 0xf0) >> 4;
 }
 
-static inline int ICACHE_FLASH_ATTR mqtt_get_dup(uint8_t* buffer) {
+static inline int ICACHE_FLASH_ATTR mqtt_get_dup(const uint8_t* buffer) {
   return (buffer[0] & 0x08) >> 3;
 }
 
-static inline int ICACHE_FLASH_ATTR mqtt_get_qos(uint8_t* buffer) {
+static inline int ICACHE_FLASH_ATTR mqtt_get_qos(const uint8_t* buffer) {
   return (buffer[0] & 0x06) >> 1;
 }
 
-static inline int ICACHE_FLASH_ATTR mqtt_get_retain(uint8_t* buffer) {
+static inline int ICACHE_FLASH_ATTR mqtt_get_retain(const uint8_t* buffer) {
   return (buffer[0] & 0x01);
 }
 
+// Init a connection descriptor
 void mqtt_msg_init(mqtt_connection_t* connection, uint8_t* buffer, uint16_t buffer_length);
-int mqtt_get_total_length(uint8_t* buffer, uint16_t length);
-const char* mqtt_get_publish_topic(uint8_t* buffer, uint16_t* length);
-const char* mqtt_get_publish_data(uint8_t* buffer, uint16_t* length);
-uint16_t mqtt_get_id(uint8_t* buffer, uint16_t length);
 
+// Returns the total length of a message including MQTT fixed header
+int mqtt_get_total_length(const uint8_t* buffer, uint16_t length);
+
+// Return pointer to topic, length in in/out param: in=length of buffer, out=length of topic
+const char* mqtt_get_publish_topic(const uint8_t* buffer, uint16_t* length);
+
+// Return pointer to data, length in in/out param: in=length of buffer, out=length of data
+const char* mqtt_get_publish_data(const uint8_t* buffer, uint16_t* length);
+
+// Return message id
+uint16_t mqtt_get_id(const uint8_t* buffer, uint16_t length);
+
+// The following functions construct an outgoing message
 mqtt_message_t* mqtt_msg_connect(mqtt_connection_t* connection, mqtt_connect_info_t* info);
 mqtt_message_t* mqtt_msg_publish(mqtt_connection_t* connection, const char* topic, const char* data, int data_length, int qos, int retain, uint16_t* message_id);
 mqtt_message_t* mqtt_msg_puback(mqtt_connection_t* connection, uint16_t message_id);

mqtt/pktbuf.c

@@ -0,0 +1,64 @@
+// Copyright 2015 by Thorsten von Eicken, see LICENSE.txt
+
+#include <esp8266.h>
+#include "pktbuf.h"
+
+void ICACHE_FLASH_ATTR
+PktBuf_Print(PktBuf *buf) {
+  os_printf("PktBuf:");
+  for (int i=-16; i<0; i++)
+    os_printf(" %02X", ((uint8_t*)buf)[i]);
+  os_printf(" %p", buf);
+  for (int i=0; i<16; i++)
+    os_printf(" %02X", ((uint8_t*)buf)[i]);
+  os_printf("\n");
+  os_printf("PktBuf: next=%p len=0x%04x\n",
+      ((void**)buf)[-4], ((uint16_t*)buf)[-6]);
+}
+
+
+PktBuf * ICACHE_FLASH_ATTR
+PktBuf_New(uint16_t length) {
+  PktBuf *buf = os_zalloc(length+sizeof(PktBuf));
+  buf->next = NULL;
+  buf->filled = 0;
+  //os_printf("PktBuf_New: %p l=%d->%d d=%p\n",
+  //    buf, length, length+sizeof(PktBuf), buf->data);
+  return buf;
+}
+
+PktBuf * ICACHE_FLASH_ATTR
+PktBuf_Push(PktBuf *headBuf, PktBuf *buf) {
+  if (headBuf == NULL) {
+    //os_printf("PktBuf_Push: %p\n", buf);
+    return buf;
+  }
+  PktBuf *h = headBuf;
+  while (h->next != NULL) h = h->next;
+  h->next = buf;
+  //os_printf("PktBuf_Push: %p->..->%p\n", headBuf, buf);
+  return headBuf;
+}
+
+PktBuf * ICACHE_FLASH_ATTR
+PktBuf_Unshift(PktBuf *headBuf, PktBuf *buf) {
+  buf->next = headBuf;
+  //os_printf("PktBuf_Unshift: %p->%p\n", buf, buf->next);
+  return buf;
+}
+
+PktBuf * ICACHE_FLASH_ATTR
+PktBuf_Shift(PktBuf *headBuf) {
+  PktBuf *buf = headBuf->next;
+  headBuf->next = NULL;
+  //os_printf("PktBuf_Shift: (%p)->%p\n", headBuf, buf);
+  return buf;
+}
+
+PktBuf * ICACHE_FLASH_ATTR
+PktBuf_ShiftFree(PktBuf *headBuf) {
+  PktBuf *buf = headBuf->next;
+  //os_printf("PktBuf_ShiftFree: (%p)->%p\n", headBuf, buf);
+  os_free(headBuf);
+  return buf;
+}

mqtt/pktbuf.h

@@ -0,0 +1,29 @@
+// Copyright 2015 by Thorsten von Eicken, see LICENSE.txt
+
+#ifndef PKTBUF_H
+#define PKTBUF_H
+
+typedef struct PktBuf {
+  struct PktBuf *next;   // next buffer in chain
+  uint16_t      filled;  // number of bytes filled in buffer
+  uint8_t       data[0]; // data in buffer
+} PktBuf;
+
+// Allocate a new packet buffer of given length
+PktBuf *PktBuf_New(uint16_t length);
+
+// Append a buffer to the end of a packet buffer queue, returns new head
+PktBuf *PktBuf_Push(PktBuf *headBuf, PktBuf *buf);
+
+// Prepend a buffer to the beginning of a packet buffer queue, return new head
+PktBuf * PktBuf_Unshift(PktBuf *headBuf, PktBuf *buf);
+
+// Shift first buffer off queue, returns new head (not shifted buffer!)
+PktBuf *PktBuf_Shift(PktBuf *headBuf);
+
+// Shift first buffer off queue, free it, return new head
+PktBuf *PktBuf_ShiftFree(PktBuf *headBuf);
+
+void PktBuf_Print(PktBuf *buf);
+
+#endif

mqtt/proto.c

@@ -1,86 +0,0 @@
-#include "proto.h"
-
-int8_t ICACHE_FLASH_ATTR 
-PROTO_Init(PROTO_PARSER* parser, PROTO_PARSE_CALLBACK* completeCallback, uint8_t* buf, uint16_t bufSize) {
-  parser->buf = buf;
-  parser->bufSize = bufSize;
-  parser->dataLen = 0;
-  parser->callback = completeCallback;
-  parser->isEsc = 0;
-  return 0;
-}
-
-int8_t ICACHE_FLASH_ATTR 
-PROTO_ParseByte(PROTO_PARSER* parser, uint8_t value) {
-  switch (value) {
-    case 0x7D:
-      parser->isEsc = 1;
-      break;
-
-    case 0x7E:
-      parser->dataLen = 0;
-      parser->isEsc = 0;
-      parser->isBegin = 1;
-      break;
-
-    case 0x7F:
-      if (parser->callback != NULL)
-        parser->callback();
-      parser->isBegin = 0;
-      return 0;
-      break;
-
-    default:
-      if (parser->isBegin == 0) break;
-
-      if (parser->isEsc) {
-        value ^= 0x20;
-        parser->isEsc = 0;
-      }
-
-      if (parser->dataLen < parser->bufSize)
-        parser->buf[parser->dataLen++] = value;
-
-      break;
-  }
-  return -1;
-}
-
-int16_t ICACHE_FLASH_ATTR 
-PROTO_ParseRb(RINGBUF* rb, uint8_t* bufOut, uint16_t* len, uint16_t maxBufLen) {
-  uint8_t c;
-
-  PROTO_PARSER proto;
-  PROTO_Init(&proto, NULL, bufOut, maxBufLen);
-  while (RINGBUF_Get(rb, &c) == 0) {
-    if (PROTO_ParseByte(&proto, c) == 0) {
-      *len = proto.dataLen;
-      return 0;
-    }
-  }
-  return -1;
-}
-
-int16_t ICACHE_FLASH_ATTR 
-PROTO_AddRb(RINGBUF* rb, const uint8_t* packet, int16_t len) {
-  uint16_t i = 2;
-  if (RINGBUF_Put(rb, 0x7E) == -1) return -1;
-  while (len--) {
-    switch (*packet) {
-      case 0x7D:
-      case 0x7E:
-      case 0x7F:
-        if (RINGBUF_Put(rb, 0x7D) == -1) return -1;
-        if (RINGBUF_Put(rb, *packet++ ^ 0x20) == -1) return -1;
-        i += 2;
-        break;
-      default:
-        if (RINGBUF_Put(rb, *packet++) == -1) return -1;
-        i++;
-        break;
-    }
-  }
-  if (RINGBUF_Put(rb, 0x7F) == -1) return -1;
-
-  return i;
-}

mqtt/proto.h

@@ -1,21 +0,0 @@
-#ifndef _PROTO_H_
-#define	_PROTO_H_
-#include <esp8266.h>
-#include "ringbuf.h"
-
-typedef void (PROTO_PARSE_CALLBACK)();
-
-typedef struct {
-  uint8_t* buf;
-  uint16_t bufSize;
-  uint16_t dataLen;
-  uint8_t isEsc;
-  uint8_t isBegin;
-  PROTO_PARSE_CALLBACK* callback;
-} PROTO_PARSER;
-
-int8_t PROTO_Init(PROTO_PARSER* parser, PROTO_PARSE_CALLBACK* completeCallback, uint8_t* buf, uint16_t bufSize);
-int16_t PROTO_AddRb(RINGBUF* rb, const uint8_t* packet, int16_t len);
-int8_t PROTO_ParseByte(PROTO_PARSER* parser, uint8_t value);
-int16_t PROTO_ParseRb(RINGBUF* rb, uint8_t* bufOut, uint16_t* len, uint16_t maxBufLen);
-#endif

mqtt/queue.c

@@ -1,53 +0,0 @@
-/* str_queue.c
-*
-* Copyright (c) 2014-2015, Tuan PM <tuanpm at live dot com>
-* All rights reserved.
-*
-* Redistribution and use in source and binary forms, with or without
-* modification, are permitted provided that the following conditions are met:
-*
-* * Redistributions of source code must retain the above copyright notice,
-* this list of conditions and the following disclaimer.
-* * Redistributions in binary form must reproduce the above copyright
-* notice, this list of conditions and the following disclaimer in the
-* documentation and/or other materials provided with the distribution.
-* * Neither the name of Redis nor the names of its contributors may be used
-* to endorse or promote products derived from this software without
-* specific prior written permission.
-*
-* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
-* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
-* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
-* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
-* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
-* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
-* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
-* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
-* POSSIBILITY OF SUCH DAMAGE.
-*/
-#include "queue.h"
-
-void ICACHE_FLASH_ATTR 
-QUEUE_Init(QUEUE* queue, int bufferSize) {
-  queue->buf = (uint8_t*)os_zalloc(bufferSize);
-  RINGBUF_Init(&queue->rb, queue->buf, bufferSize);
-}
-
-int32_t ICACHE_FLASH_ATTR 
-QUEUE_Puts(QUEUE* queue, uint8_t* buffer, uint16_t len) {
-  return PROTO_AddRb(&queue->rb, buffer, len);
-}
-
-int32_t ICACHE_FLASH_ATTR 
-QUEUE_Gets(QUEUE* queue, uint8_t* buffer, uint16_t* len, uint16_t maxLen) {
-  return PROTO_ParseRb(&queue->rb, buffer, len, maxLen);
-}
-
-bool ICACHE_FLASH_ATTR 
-QUEUE_IsEmpty(QUEUE* queue) {
-  if (queue->rb.fill_cnt <= 0)
-    return TRUE;
-  return FALSE;
-}

mqtt/queue.h

@@ -1,46 +0,0 @@
-/* str_queue.h --
-*
-* Copyright (c) 2014-2015, Tuan PM <tuanpm at live dot com>
-* All rights reserved.
-*
-* Redistribution and use in source and binary forms, with or without
-* modification, are permitted provided that the following conditions are met:
-*
-* * Redistributions of source code must retain the above copyright notice,
-* this list of conditions and the following disclaimer.
-* * Redistributions in binary form must reproduce the above copyright
-* notice, this list of conditions and the following disclaimer in the
-* documentation and/or other materials provided with the distribution.
-* * Neither the name of Redis nor the names of its contributors may be used
-* to endorse or promote products derived from this software without
-* specific prior written permission.
-*
-* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
-* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
-* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
-* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
-* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
-* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
-* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
-* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
-* POSSIBILITY OF SUCH DAMAGE.
-*/
-
-#ifndef USER_QUEUE_H_
-#define USER_QUEUE_H_
-#include <esp8266.h>
-#include "proto.h"
-#include "ringbuf.h"
-
-typedef struct {
-  uint8_t* buf;
-  RINGBUF rb;
-} QUEUE;
-
-void QUEUE_Init(QUEUE* queue, int bufferSize);
-int32_t QUEUE_Puts(QUEUE* queue, uint8_t* buffer, uint16_t len);
-int32_t QUEUE_Gets(QUEUE* queue, uint8_t* buffer, uint16_t* len, uint16_t maxLen);
-bool QUEUE_IsEmpty(QUEUE* queue);
-#endif /* USER_QUEUE_H_ */

mqtt/ringbuf.c

@@ -1,63 +0,0 @@
-#include "ringbuf.h"
-
-/**
-* \brief init a RINGBUF object
-* \param r pointer to a RINGBUF object
-* \param buf pointer to a byte array
-* \param size size of buf
-* \return 0 if successfull, otherwise failed
-*/
-int16_t ICACHE_FLASH_ATTR 
-RINGBUF_Init(RINGBUF* r, uint8_t* buf, int32_t size) {
-  if (r == NULL || buf == NULL || size < 2) return -1;
-
-  r->p_o = r->p_r = r->p_w = buf;
-  r->fill_cnt = 0;
-  r->size = size;
-
-  return 0;
-}
-
-/**
-* \brief put a character into ring buffer
-* \param r pointer to a ringbuf object
-* \param c character to be put
-* \return 0 if successfull, otherwise failed
-*/
-int16_t ICACHE_FLASH_ATTR 
-RINGBUF_Put(RINGBUF* r, uint8_t c) {
-  if (r->fill_cnt >= r->size)return -1; // ring buffer is full, this should be atomic operation
-
-
-  r->fill_cnt++; // increase filled slots count, this should be atomic operation
-
-
-  *r->p_w++ = c; // put character into buffer
-
-  if (r->p_w >= r->p_o + r->size) // rollback if write pointer go pass
-    r->p_w = r->p_o; // the physical boundary
-
-  return 0;
-}
-
-/**
-* \brief get a character from ring buffer
-* \param r pointer to a ringbuf object
-* \param c read character
-* \return 0 if successfull, otherwise failed
-*/
-int16_t ICACHE_FLASH_ATTR 
-RINGBUF_Get(RINGBUF* r, uint8_t* c) {
-  if (r->fill_cnt <= 0)return -1; // ring buffer is empty, this should be atomic operation
-
-
-  r->fill_cnt--; // decrease filled slots count
-
-
-  *c = *r->p_r++; // get the character out
-
-  if (r->p_r >= r->p_o + r->size) // rollback if write pointer go pass
-    r->p_r = r->p_o; // the physical boundary
-
-  return 0;
-}

mqtt/ringbuf.h

@@ -1,17 +0,0 @@
-#ifndef _RING_BUF_H_
-#define _RING_BUF_H_
-
-#include <esp8266.h>
-
-typedef struct {
-  uint8_t* p_o; /**< Original pointer */
-  uint8_t* volatile p_r; /**< Read pointer */
-  uint8_t* volatile p_w; /**< Write pointer */
-  volatile int32_t fill_cnt; /**< Number of filled slots */
-  int32_t size; /**< Buffer size */
-} RINGBUF;
-
-int16_t RINGBUF_Init(RINGBUF* r, uint8_t* buf, int32_t size);
-int16_t RINGBUF_Put(RINGBUF* r, uint8_t c);
-int16_t RINGBUF_Get(RINGBUF* r, uint8_t* c);
-#endif

rest/rest.c

@@ -154,12 +154,12 @@ rest_dns_found(const char *name, ip_addr_t *ipaddr, void *arg) {
 
   if(client->ip.addr == 0 && ipaddr->addr != 0) {
     os_memcpy(client->pCon->proto.tcp->remote_ip, &ipaddr->addr, 4);
-    //if(client->security){
+#ifdef CLIENT_SSL_ENABLE
-    //  espconn_secure_connect(client->pCon);
+    if(client->security) {
-    //}
+      espconn_secure_connect(client->pCon);
-    //else {
+    } else
+#endif
       espconn_connect(client->pCon);
-    //}
     os_printf("REST: connecting...\n");
   }
 }
@@ -368,6 +368,8 @@ REST_Request(CmdPacket *cmd) {
   }
   os_printf("\n");
 
+  //os_printf("REST request: %s", (char*)client->data);
+
   os_printf("REST: pCon state=%d\n", client->pCon->state);
   client->pCon->state = ESPCONN_NONE;
   espconn_regist_connectcb(client->pCon, tcpclient_connect_cb);

serial/serbridge.c

@@ -14,6 +14,7 @@
 #include "serled.h"
 #include "config.h"
 #include "console.h"
+#include "slip.h"
 #include "cmd.h"
 
 static struct espconn serbridgeConn;
@@ -258,6 +259,19 @@ console_process(char *buf, short len) {
   }
 }
 
+// callback with a buffer of characters that have arrived on the uart
+void ICACHE_FLASH_ATTR
+serbridgeUartCb(char *buf, short length) {
+  if (!flashConfig.slip_enable || slip_disabled > 0) {
+    //os_printf("SLIP: disabled got %d\n", length);
+    console_process(buf, length);
+  } else {
+    slip_parse_buf(buf, length);
+  }
+
+  serledFlash(50); // short blink on serial LED
+}
+
 //callback after the data are sent
 static void ICACHE_FLASH_ATTR
 serbridgeSentCb(void *arg) {
@@ -273,6 +287,7 @@ serbridgeSentCb(void *arg) {
 
 // Error callback (it's really poorly named, it's not a "connection reconnected" callback,
 // it's really a "connection broken, please reconnect" callback)
+// Note that there is no DisconCb after a ReconCb
 static void ICACHE_FLASH_ATTR serbridgeReconCb(void *arg, sint8 err) {
   serbridgeConnData *sbConn = serbridgeFindConnData(arg);
   if (sbConn == NULL) return;

serial/serled.h

@@ -6,4 +6,3 @@ void serledInit(void);
 void makeGpio(uint8_t pin);
 
 #endif
-

serial/slip.c

@@ -4,11 +4,10 @@
 #include "uart.h"
 #include "crc16.h"
 #include "serbridge.h"
-#include "serled.h"
 #include "console.h"
 #include "cmd.h"
 
-uint8_t slip_disabled;   // disable slip to allow flashing of attached MCU
+uint8_t slip_disabled;   // temporarily disable slip to allow flashing of attached MCU
 
 extern void ICACHE_FLASH_ATTR console_process(char *buf, short len);
 
@@ -115,18 +114,7 @@ slip_parse_char(char c) {
 
 // callback with a buffer of characters that have arrived on the uart
 void ICACHE_FLASH_ATTR
-serbridgeUartCb(char *buf, short length) {
+slip_parse_buf(char *buf, short length) {
-  if (slip_disabled > 0) {
-    //os_printf("SLIP: disabled got %d\n", length);
-    console_process(buf, length);
-    for (short i=0; i<length; i++)
-      if (buf[i] == SLIP_START) {
-        os_printf("SLIP: START while disabled=%d\n", slip_disabled);
-        break;
-      }
-    return;
-  }
-
   // do SLIP parsing
   for (short i=0; i<length; i++)
     slip_parse_char(buf[i]);
@@ -136,6 +124,5 @@ serbridgeUartCb(char *buf, short length) {
     slip_process();
     slip_reset();
   }
-
-  serledFlash(50); // short blink on serial LED
 }
+

serial/slip.h

@@ -0,0 +1,6 @@
+#ifndef SLIP_H
+#define SLIP_H
+
+void slip_parse_buf(char *buf, short length);
+
+#endif

serial/uart.c

@@ -23,7 +23,7 @@
 #include "user_interface.h"
 #include "uart.h"
 
-#define recvTaskPrio        0
+#define recvTaskPrio        1
 #define recvTaskQueueLen    64
 
 // UartDev is defined and initialized in rom code.

user/user_main.c

@@ -1,70 +1,87 @@
 #include <esp8266.h>
-//#include <mqtt.h>
+#include "cgiwifi.h"
-//#include <cgiwifi.h>
+#include "mqtt.h"
 
-//MQTT_Client mqttClient;
+MQTT_Client mqttClient;
-//
+
-//void ICACHE_FLASH_ATTR 
+static ETSTimer mqttTimer;
-//mqttConnectedCb(uint32_t *args) {
+
+static int once = 0;
+static void ICACHE_FLASH_ATTR mqttTimerCb(void *arg) {
+  if (once++ > 0) return;
+  MQTT_Init(&mqttClient, "h.voneicken.com", 1883, 0, 2, "test1", "", "", 10, 1);
+  MQTT_Connect(&mqttClient);
+  MQTT_Subscribe(&mqttClient, "system/time", 0);
+}
+
+void ICACHE_FLASH_ATTR
+wifiStateChangeCb(uint8_t status)
+{
+  if (status == wifiGotIP) {
+    os_timer_disarm(&mqttTimer);
+    os_timer_setfn(&mqttTimer, mqttTimerCb, NULL);
+    os_timer_arm(&mqttTimer, 15000, 0);
+  }
+}
+
+
+// initialize the custom stuff that goes beyond esp-link
+void app_init() {
+  wifiAddStateChangeCb(wifiStateChangeCb);
+}
+
+
+#if 0
+MQTT_Client mqttClient;
+
+void ICACHE_FLASH_ATTR
+mqttConnectedCb(uint32_t *args) {
+  MQTT_Client* client = (MQTT_Client*)args;
+  MQTT_Publish(client, "announce/all", "Hello World!", 0, 0);
+}
+
+void ICACHE_FLASH_ATTR
+mqttDisconnectedCb(uint32_t *args) {
 //  MQTT_Client* client = (MQTT_Client*)args;
-//  MQTT_Publish(client, "announce/all", "Hello World!", 0, 0);
+  os_printf("MQTT Disconnected\n");
-//}
-//
-//void ICACHE_FLASH_ATTR 
-//mqttDisconnectedCb(uint32_t *args) {
-////  MQTT_Client* client = (MQTT_Client*)args;
-//  os_printf("MQTT Disconnected\n");
-//}
-//
-//void ICACHE_FLASH_ATTR 
-//mqttTcpDisconnectedCb(uint32_t *args) {
-////  MQTT_Client* client = (MQTT_Client*)args;
-//  os_printf("MQTT TCP Disconnected\n");
-//}
-//
-//void ICACHE_FLASH_ATTR 
-//mqttPublishedCb(uint32_t *args) {
-////  MQTT_Client* client = (MQTT_Client*)args;
-//  os_printf("MQTT Published\n");
-//}
-//
-//void ICACHE_FLASH_ATTR 
-//mqttDataCb(uint32_t *args, const char* topic, uint32_t topic_len, const char *data, uint32_t data_len) {
-//  char *topicBuf = (char*)os_zalloc(topic_len + 1);
-//  char *dataBuf = (char*)os_zalloc(data_len + 1);
-//
-////  MQTT_Client* client = (MQTT_Client*)args;
-//
-//  os_memcpy(topicBuf, topic, topic_len);
-//  topicBuf[topic_len] = 0;
-//
-//  os_memcpy(dataBuf, data, data_len);
-//  dataBuf[data_len] = 0;
-//
-//  os_printf("Receive topic: %s, data: %s\n", topicBuf, dataBuf);
-//  os_free(topicBuf);
-//  os_free(dataBuf);
-//}
-//
-//void ICACHE_FLASH_ATTR 
-//wifiStateChangeCb(uint8_t status)
-//{
-//  if (status == wifiGotIP  && mqttClient.connState != TCP_CONNECTING){
-//    MQTT_Connect(&mqttClient);
-//  }
-//  else if (status == wifiIsDisconnected && mqttClient.connState == TCP_CONNECTING){    
-//    MQTT_Disconnect(&mqttClient);
-//  }
-//}
-
-void init() {
-//  wifiAddStateChangeCb(wifiStateChangeCb);
-//  MQTT_InitConnection(&mqttClient, MQTT_HOST, MQTT_PORT, MQTT_SECURITY);
-//  MQTT_InitClient(&mqttClient, MQTT_CLIENT_ID, MQTT_USER, MQTT_PASS, MQTT_KEEPALIVE, MQTT_CLSESSION);
-//  MQTT_InitLWT(&mqttClient, "/lwt", "offline", 0, 0);
-//  MQTT_OnConnected(&mqttClient, mqttConnectedCb);
-//  MQTT_OnDisconnected(&mqttClient, mqttDisconnectedCb);
-//  MQTT_OnDisconnected(&mqttClient, mqttTcpDisconnectedCb);
-//  MQTT_OnPublished(&mqttClient, mqttPublishedCb);
-//  MQTT_OnData(&mqttClient, mqttDataCb);
 }
+
+void ICACHE_FLASH_ATTR
+mqttTcpDisconnectedCb(uint32_t *args) {
+//  MQTT_Client* client = (MQTT_Client*)args;
+  os_printf("MQTT TCP Disconnected\n");
+}
+
+void ICACHE_FLASH_ATTR
+mqttPublishedCb(uint32_t *args) {
+//  MQTT_Client* client = (MQTT_Client*)args;
+  os_printf("MQTT Published\n");
+}
+
+void ICACHE_FLASH_ATTR
+mqttDataCb(uint32_t *args, const char* topic, uint32_t topic_len, const char *data, uint32_t data_len) {
+  char *topicBuf = (char*)os_zalloc(topic_len + 1);
+  char *dataBuf = (char*)os_zalloc(data_len + 1);
+
+//  MQTT_Client* client = (MQTT_Client*)args;
+
+  os_memcpy(topicBuf, topic, topic_len);
+  topicBuf[topic_len] = 0;
+
+  os_memcpy(dataBuf, data, data_len);
+  dataBuf[data_len] = 0;
+
+  os_printf("Receive topic: %s, data: %s\n", topicBuf, dataBuf);
+  os_free(topicBuf);
+  os_free(dataBuf);
+}
+
+  MQTT_InitConnection(&mqttClient, MQTT_HOST, MQTT_PORT, MQTT_SECURITY);
+  MQTT_InitClient(&mqttClient, MQTT_CLIENT_ID, MQTT_USER, MQTT_PASS, MQTT_KEEPALIVE, MQTT_CLSESSION);
+  MQTT_InitLWT(&mqttClient, "/lwt", "offline", 0, 0);
+  MQTT_OnConnected(&mqttClient, mqttConnectedCb);
+  MQTT_OnDisconnected(&mqttClient, mqttDisconnectedCb);
+  MQTT_OnDisconnected(&mqttClient, mqttTcpDisconnectedCb);
+  MQTT_OnPublished(&mqttClient, mqttPublishedCb);
+  MQTT_OnData(&mqttClient, mqttDataCb);
+#endif