Merge pull request #1 from blackcoffeexbt/bugfix/timeout-at-least-one-relay-needed

blackcoffeexbt · web-flow · commit 3f225b6737f3 · 2023-04-06T16:25:49.000+01:00
We need at least one relay connected even if timeout is reached
diff --git a/src/NostrRelayManager.cpp b/src/NostrRelayManager.cpp
@@ -110,7 +110,7 @@ void NostrRelayManager::broadcastEvents() {
     }
 
     lastBroadcastAttempt = currentMillis;
-  } else if (currentMillis - lastBroadcastAttempt > minRelaysTimeout) {
+  } else if (currentMillis - lastBroadcastAttempt > minRelaysTimeout && connectedRelayCount() > 0) {
     // Broadcast all queued events to all relays, regardless of whether they're connected
     while (!m_queue.isEmpty()) {
       const char *item = m_queue.dequeue();

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ void NostrRelayManager::broadcastEvents() {`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`lastBroadcastAttempt = currentMillis;`
`113`		`- } else if (currentMillis - lastBroadcastAttempt > minRelaysTimeout) {`
	`113`	`+ } else if (currentMillis - lastBroadcastAttempt > minRelaysTimeout && connectedRelayCount() > 0) {`
`114`	`114`	`// Broadcast all queued events to all relays, regardless of whether they're connected`
`115`	`115`	`while (!m_queue.isEmpty()) {`
`116`	`116`	`const char *item = m_queue.dequeue();`