Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Socket not connected' incorrectly leads to FAILED connection state rather than DISCONNECTED #471

Closed
SimonWoolf opened this issue Aug 26, 2016 · 1 comment

Comments

@SimonWoolf
Copy link
Member

SimonWoolf commented Aug 26, 2016

From https://app.intercom.io/a/apps/ua39m1ld/inbox/all/conversations/5776825348, happens if they disconnect from the internet.

2016-08-26 16:22:43.661 GetSetClean[5322:345395] DEBUG: (ARTWebSocketTransport.m:295) R:0x7f8f4d8e5ab0 WS:0x7f8f4b6b0f90 websocket did receive error Error Domain=NSPOSIXErrorDomain Code=57 "Socket is not connected"
2016-08-26 16:22:43.662 GetSetClean[5322:345395] DEBUG: (ARTRealtime.m:756) R:0x7f8f4d8e5ab0 realtime transport failed: <ARTRealtimeTransportError: 0x7f8f4b06a090 {
type: Other
badResponseCode: 0
url: /?key=AvM5bQ.oXRTGw:nRccq-fq_HBE05WQ&echo=true&format=msgpack&v=0.8 -- wss://realtime.ably.io:443
error: Error Domain=NSPOSIXErrorDomain Code=57 "Socket is not connected"
}>
2016-08-26 16:22:43.663 GetSetClean[5322:345395] DEBUG: (ARTRealtime.m:212) R:0x7f8f4d8e5ab0 transition to Failed requested
2016-08-26 16:22:43.663 GetSetClean[5322:345395] failed! ARTErrorInfo with code 0, message: Error Domain=NSPOSIXErrorDomain Code=57 "Socket is not connected"
2016-08-26 16:22:43.664 GetSetClean[5322:345395] DEBUG: (ARTWebSocketTransport.m:62) R:0x0 WS:0x7f8f4b6b0f90 dealloc
@tcard
Copy link
Contributor

tcard commented Aug 26, 2016

Turns out I already fixed in #444 and git just happily merged the two identical-except-for-line-order fixes. I reverted #472.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants