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: Add peer closed WebSocket handling introduced in StarScream 4.0.5 #162

Closed
wants to merge 8 commits into from

Conversation

Fedos
Copy link
Contributor

@Fedos Fedos commented Sep 28, 2023

RFC 6455 7.1.5 says:

   If _The WebSocket
   Connection is Closed_ and no Close control frame was received by the
   endpoint (such as could occur if the underlying transport connection
   is lost), _The WebSocket Connection Close Code_ is considered to be
   1006

Fedos and others added 8 commits June 30, 2023 11:23
Call it before app resigns active, so OS won't abort the connection like this:
Receive failed with error "Software caused connection abort"
…eric parameter shadowing generic parameter from outer scope with the same name
…5. According to RFC 6455

RFC 6455 7.1.5 says:
   If _The WebSocket
   Connection is Closed_ and no Close control frame was received by the
   endpoint (such as could occur if the underlying transport connection
   is lost), _The WebSocket Connection Close Code_ is considered to be
   1006
@vercel
Copy link

vercel bot commented Sep 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
swift-graphql ❌ Failed (Inspect) Sep 28, 2023 2:58pm

@Fedos
Copy link
Contributor Author

Fedos commented Sep 28, 2023

@maticzav This looks to me as a breaking change if someone, who uses WebSocket has a switch without a default case.
But TBH, it is already broken for everyone using it, because the library itself also lacks aforementioned default case.

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.

None yet

1 participant