From 56209223f2d46713030a9f45b1c36e5dd6242a89 Mon Sep 17 00:00:00 2001
From: Skot <skot@bitnet.cx>
Date: Wed, 5 Jun 2024 17:31:35 -0400
Subject: [PATCH] moved the DNS lookup inside the stratum connection retry loop
 (#204)

---
 main/tasks/stratum_task.c | 45 ++++++++++++++++++++++-----------------
 1 file changed, 25 insertions(+), 20 deletions(-)

diff --git a/main/tasks/stratum_task.c b/main/tasks/stratum_task.c
index 949044ce3..751386827 100644
--- a/main/tasks/stratum_task.c
+++ b/main/tasks/stratum_task.c
@@ -48,30 +48,35 @@ void stratum_task(void * pvParameters)
     char *stratum_url = GLOBAL_STATE->SYSTEM_MODULE.pool_url;
     uint16_t port = GLOBAL_STATE->SYSTEM_MODULE.pool_port;
 
-    // check to see if the STRATUM_URL is an ip address already
-    if (inet_pton(AF_INET, stratum_url, &ip_Addr) == 1) {
-        bDNSFound = true;
-    }
-    else
-    {
-        ESP_LOGI(TAG, "Get IP for URL: %s\n", stratum_url);
-        dns_gethostbyname(stratum_url, &ip_Addr, dns_found_cb, NULL);
-        while (!bDNSFound);
-
-        if (bDNSInvalid) {
-            ESP_LOGE(TAG, "DNS lookup failed for URL: %s\n", stratum_url);
-            //set ip_Addr to 0.0.0.0 so that connect() will fail
-            IP_ADDR4(&ip_Addr, 0, 0, 0, 0);
+    while (1) {
+        //clear flags used by the dns callback, dns_found_cb()
+        bDNSFound = false;
+        bDNSInvalid = false;
+
+        // check to see if the STRATUM_URL is an ip address already
+        if (inet_pton(AF_INET, stratum_url, &ip_Addr) == 1) {
+            bDNSFound = true;
         }
+        else
+        {
+            ESP_LOGI(TAG, "Get IP for URL: %s", stratum_url);
+            dns_gethostbyname(stratum_url, &ip_Addr, dns_found_cb, NULL);
+            while (!bDNSFound);
+
+            if (bDNSInvalid) {
+                ESP_LOGE(TAG, "DNS lookup failed for URL: %s", stratum_url);
+                //set ip_Addr to 0.0.0.0 so that connect() will fail
+                IP_ADDR4(&ip_Addr, 0, 0, 0, 0);
+            }
 
-    }
+        }
 
-    // make IP address string from ip_Addr
-    snprintf(host_ip, sizeof(host_ip), "%d.%d.%d.%d", ip4_addr1(&ip_Addr.u_addr.ip4), ip4_addr2(&ip_Addr.u_addr.ip4),
-             ip4_addr3(&ip_Addr.u_addr.ip4), ip4_addr4(&ip_Addr.u_addr.ip4));
-    ESP_LOGI(TAG, "Connecting to: stratum+tcp://%s:%d (%s)\n", stratum_url, port, host_ip);
+        // make IP address string from ip_Addr
+        snprintf(host_ip, sizeof(host_ip), "%d.%d.%d.%d", ip4_addr1(&ip_Addr.u_addr.ip4), ip4_addr2(&ip_Addr.u_addr.ip4),
+                    ip4_addr3(&ip_Addr.u_addr.ip4), ip4_addr4(&ip_Addr.u_addr.ip4));
+        ESP_LOGI(TAG, "Connecting to: stratum+tcp://%s:%d (%s)", stratum_url, port, host_ip);
 
-    while (1) {
+    
         struct sockaddr_in dest_addr;
         dest_addr.sin_addr.s_addr = inet_addr(host_ip);
         dest_addr.sin_family = AF_INET;