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

Update WebSocketConnecting to match latest starscream #1205

Open
simonmcl opened this issue Nov 1, 2023 · 4 comments
Open

Update WebSocketConnecting to match latest starscream #1205

simonmcl opened this issue Nov 1, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@simonmcl
Copy link

simonmcl commented Nov 1, 2023

Is your feature request related to a problem? Please describe.
I didn't want to rely on starscream for my app, but ran into issues integrating my own socket, so I gave in and updated my app to use the same setup as your example app. Now i'm getting crash reports from the starscream version used in the example project, when it tries to set http headers. I looked into it and noticed your example app is using a fairly old version of starscream and the supplied protocol WebSocketConnecting doesn't match the latest interface.

Describe the solution you'd like
Can you either update WebSocketConnecting so that the latest starscream can be plugged in more easily, or can you provide docs/sample for using the latest. E.g. updating the example to starscream v4+

Describe alternatives you've considered
I tired building my own websocket, but i'm having a lot of issues keeping the connection open. By default the inbuilt iOS sockets close after 5 minutes of no messages. I'm having a lot of issues trying to balance the reconnections with users trying to make use of it. The ideal solution here is for WC2 to implement a custom hidden keepalive message, thats sent on a regular basis to keep the socket open, rather than relying on reconnection. Websocket implementations vary wildly across platforms, custom keepalives has been the only reliable means of keeping connections open in various past projects/companies

@simonmcl simonmcl added the enhancement New feature or request label Nov 1, 2023
@2jumper3
Copy link
Contributor

2jumper3 commented Nov 2, 2023

How to vote for this enhancement?)
I believe we are on the same page regarding this matter.

https://github.com/orgs/WalletConnect/discussions/3857

@llbartekll
Copy link
Contributor

E.g. updating the example to starscream v4+

The default interface matches Starscream v3 because we found it most stable.

but we are currently testing internally URLSession solution, #911
@simonmcl I would appreciate if you could give your feedback.

@alexanderkhitev
Copy link

hello @llbartekll ! Any news/updates?

@alexanderkhitev
Copy link

hello! Any news/updates? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants