Removed attempt code from get/post. Examples reset.

Author: Melissa LeBlanc-Williams

adafruit_esp32spi/adafruit_esp32spi_wifimanager.py

@@ -39,11 +39,11 @@ class ESPSPI_WiFiManager:
     """
     A class to help manage the Wifi connection
     """
-    def __init__(self, esp, settings, status_neopixel=None, attempts=1):
+    def __init__(self, esp, settings, status_neopixel=None, attempts=2):
         """
         :param ESP_SPIcontrol esp: The ESP object we are using
         :param dict settings: The WiFi and Adafruit IO Settings (See examples)
-        :param int attempts: (Optional) Failed attempts before resetting the ESP32 (default=1)
+        :param int attempts: (Optional) Failed attempts before resetting the ESP32 (default=2)
         :param status_neopixel: (Optional) The neopixel pin - Usually board.NEOPIXEL (default=None)
         :type status_neopixel: Pin
         """
@@ -60,6 +60,14 @@ def __init__(self, esp, settings, status_neopixel=None, attempts=1):
             self.neopix = None
         self.neo_status(0)
 
+    def reset(self):
+        """
+        Perform a hard reset on the ESP32
+        """
+        if self.debug:
+            print("Resetting ESP32\n", error)
+        self._esp.reset()
+
     def connect(self):
         """
         Attempt to connect to WiFi using the current settings
@@ -85,8 +93,7 @@ def connect(self):
                 failure_count += 1
                 if failure_count >= self.attempts:
                     failure_count = 0
-                    self._esp.reset()
-                    print("Resetting ESP32\n", error)
+                    self.reset()
                 continue
 
     def get(self, url, **kw):
@@ -104,16 +111,7 @@ def get(self, url, **kw):
         if not self._esp.is_connected:
             self.connect()
         self.neo_status((100, 100, 0))
-        attempt_count = 0
-        while attempt_count < self.attempts:
-            try:
-                attempt_count += 1
-                return_val = requests.get(url, **kw)
-            except(ValueError, RuntimeError) as error:
-                if attempt_count >= self.attempts:
-                    attempt_count = 0
-                    self._esp.reset()
-                    print("Resetting ESP32\n", error)
+        return_val = requests.get(url, **kw)
         self.neo_status(0)
         return return_val
 
@@ -132,17 +130,7 @@ def post(self, url, **kw):
         if not self._esp.is_connected:
             self.connect()
         self.neo_status((100, 100, 0))
-        attempt_count = 0
-        while attempt_count < self.attempts:
-            try:
-                attempt_count += 1
-                return_val = requests.post(url, **kw)
-            except(ValueError, RuntimeError) as error:
-                if attempt_count >= self.attempts:
-                    attempt_count = 0
-                    self._esp.reset()
-                    print("Resetting ESP32\n", error)
-        self.neo_status(0)
+        return_val = requests.post(url, **kw)
         return return_val
 
     def put(self, url, **kw):

examples/esp32spi_aio_post.py

@@ -38,6 +38,7 @@
         print("OK")
     except (ValueError, RuntimeError) as e:
         print("Failed to get data, retrying\n", e)
+        wifi.reset()
         continue
     response = None
     time.sleep(15)

examples/esp32spi_cheerlights.py

@@ -47,6 +47,7 @@
         response.close()
     except (ValueError, RuntimeError) as e:
         print("Failed to get data, retrying\n", e)
+        wifi.reset()
         continue
 
     if not value: