tyiu/damus
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't put reconnect logic in ping

Browse Source 
It seems to be very bad
This commit is contained in:
William Casarin
2023-05-07 11:17:14 -07:00
parent b43aca8abb
commit 8c55f23ea8
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
damus/Nostr/RelayConnection.swift
View File

@@ -57,11 +57,7 @@ final class RelayConnection {
func ping() { func ping() {
socket.ping { err in socket.ping { err in
if let err { if err != nil {
self.isConnecting = false
self.isConnected = false
self.reconnect_with_backoff()
} else {
self.last_pong = .now self.last_pong = .now
} }
} }
@@ -124,7 +120,7 @@ final class RelayConnection {
DispatchQueue.main.async { DispatchQueue.main.async {
self.isConnected = false self.isConnected = false
self.isConnecting = false self.isConnecting = false
self.reconnect_with_backoff() self.reconnect()
} }
case .error(let error): case .error(let error):
print("⚠️ Warning: RelayConnection (\(self.url)) error: \(error)") print("⚠️ Warning: RelayConnection (\(self.url)) error: \(error)")