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

Triggered reconnect on 3000 WebSocket error #13

Closed
wants to merge 1 commit into from

Conversation

cdanielw
Copy link

When connection is lost and resumed, the client can receive a c[3000,"No response from heartbeat"] message. This causes the client to disconnect. This commit treats the 3000 error as a recoverable error, and try to reconnect.

This issue is triggered in Google Chrome, which doesn't send pings to check if connection is healthy. Safari and Firefox notice a dropped connection before this situation occurs.

@aronatkins aronatkins requested a review from jcheng5 June 25, 2018 13:23
@jcheng5
Copy link
Member

jcheng5 commented Jun 28, 2018

Thanks, good call!

Sorry for the inconvenience, but to accept contributions to RStudio packages, we require a signed individual or corporate contributor agreement as appropriate. You can send the signed copy to jj@rstudio.com. (Once you've signed, you're good across all RStudio repos)

@jcheng5
Copy link
Member

jcheng5 commented May 20, 2020

Yikes, this is still open.

We can do without the CLA for such a simple change, but the dist/ files have changed a lot more than they should have--we should rebuild ourselves.

@aronatkins
Copy link
Collaborator

Replaced by #25.

@aronatkins aronatkins closed this Mar 23, 2021
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

3 participants