Added extra check to avoid invoking goroutine on empty internal members

ably · Nov 21, 2023 · ca2eed1 · ca2eed1
1 parent f7afecd
commit ca2eed1
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/ably/realtime_presence.go b/ably/realtime_presence.go
@@ -175,13 +175,15 @@ func (pres *RealtimePresence) onAttach(msg *protocolMessage, isAttachWithoutMess
 	pres.queue.Flush() // RTP5b
 	// RTP17f
 	if isAttachWithoutMessageLoss {
-		internalMembers := make([]*PresenceMessage, len(pres.internalMembers))
-		indexCounter := 0
-		for _, member := range pres.internalMembers {
-			internalMembers[indexCounter] = member
-			indexCounter = indexCounter + 1
+		if len(pres.internalMembers) > 0 {
+			internalMembers := make([]*PresenceMessage, len(pres.internalMembers))
+			indexCounter := 0
+			for _, member := range pres.internalMembers {
+				internalMembers[indexCounter] = member
+				indexCounter = indexCounter + 1
+			}
+			go pres.enterMembers(internalMembers)
 		}
-		go pres.enterMembers(internalMembers)
 	}
 }