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

fix: refresh #9

Open
wants to merge 69 commits into
base: master
Choose a base branch
from
Open

fix: refresh #9

wants to merge 69 commits into from

Conversation

CreatFish
Copy link

No description provided.

dsrees and others added 30 commits October 11, 2021 09:38
Unwrapping response from message payload
Fixes receive callbacks not being triggered
switched the foreground observation for the more reliable didBecomeAc…
Added use of product to use rxswift or starscream
…ift-dependencies

Removing RxSwift and Starscream as dependencies
Fix not always automatically reconnecting after hearbeat timeout
Added support for goingAway to temporarily close a socket
…rror-callback

Added URLResponse to socket.onError to check for status codes
ejensen and others added 30 commits April 5, 2023 13:05
Using SynchronizedArray to achieve threadsafe access to bindingsDel
Add code and reason to socket close callbacks
* Pass response in `onOpen` callback

The response may be useful for the clients for diagnosis and observability.
E.g. it makes it possible to extract a Cloudflare Ray id from the HTTP headers.

`onOpen` methods with no value passed to the callback are preserved for backward compatibility.

* Remove redundant tuple packaging

Signed-off-by: Daniel Garbień <[email protected]>

---------

Signed-off-by: Daniel Garbień <[email protected]>
Signed-off-by: Daniel Garbień <[email protected]>
Added headers to be sent to Websocket
Update existing SynchronizedArray instances instead of replacing the instances
Prevent crashes caused by threading issues
Prevent crash caused by non-exclusive access to the Channels array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants