Leaked connections when trying to connect to a high latency server

[jlcvp]

This following snippet of code in the project is creating multiple connections when connecting to high latency servers. The while loop calls self.client?.connect again (thus, creating another websocketclient) if the first call takes more than 200 + self._checksDelay millis to connect/fail

Action-Cable-Swift/Sources/ActionCableSwift/Helpers.swift

Lines 155 to 162 in ebb9986

	if !self.isConnected() {
	self.client?.disconnect()
	usleep(200_000)
	self.client?.connect()
	usleep(self._checksDelay)
	self.updateLastPoint()
	continue
	}

The problem with this busy wait approach is that we don't have a way outside this library code to manage those connections and terminate the duplicates if needed.

I think we Should try to use a Lock before the if !self.isConnected() { check to do this verification only after the connection handshake itself. That way we would have only 2 states possible (connection successful or connection failed), hence avoiding the duplicates

[ankitpoplify]

@jlcvp I'm trying to test this library with one of my projects and I found that it is creating 6 connections at a time. Is this because of this code snippet?

[nerzh]

I think we need to remove the reconnect code. I will remove it and let the ios developer himself be responsible for this

[ankitpoplify]

@nerzh Which reconnect code?

[nerzh]

disable reconnect

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 35 in ebb9986

if self.options.reconnect {

passing the parameter false.
Next, implement the logic that suits you. You will need to monitor the connection and, in case of a break, reconnect manually by calling the connect method.

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 72 in ebb9986

public func connect() {

[ankitpoplify]

I’m using this for live chat, can you please give some basic idea for how to implement logic to monitor connection?

[nerzh]

you have a bunch of connection lifecycle callbacks. Implement your logic, what to do if the connection is broken or for some reason the ping stops coming, etc. All servers are different, they work differently, so it’s difficult to write a universal reconnection. At the moment you say you have 6 connections, I don't know how you counted that, but if that's the case, then implement the reconnection logic yourself that will do what you want. To do this, handle all situations in which the connection was interrupted

[nerzh]

these callbacks should help you

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 44 in ebb9986

public func addOnDisconnected(_ handler: @escaping (_ reason: String?) -> Void) {

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 48 in ebb9986

public func addOnCancelled(_ handler: @escaping () -> Void) {

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 60 in ebb9986

public func addOnPing(_ handler: @escaping () -> Void) {

Action-Cable-Swift/Sources/ActionCableSwift/ActionCableSwift.swift

Line 64 in ebb9986

public func addOnPong(_ handler: @escaping () -> Void) {