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: update to 7.5.1 of rpc-websockets; fixes socket state corruption related to automatic reconnections #1180

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

steveluscher
Copy link
Collaborator

At long last, we've found a clear-cut source of socket state corruption in rpc-websockets and were able to fix it in elpheria/rpc-websockets#138.

This pulls the new version into @solana/web3.js. Now, when our library make an explicit reconnection attempt, it will no longer cause the socket state to get corrupted by rpc-websocket's implicit reconnection machinery.

Fixes #1106.

@codecov-commenter
Copy link

codecov-commenter commented Feb 23, 2023

Codecov Report

Merging #1180 (8ce5f21) into master (33429ca) will increase coverage by 0.12%.
The diff coverage is n/a.

❗ Current head 8ce5f21 differs from pull request most recent head c0c79d6. Consider uploading reports for the commit c0c79d6 to get more accurate results

@@            Coverage Diff             @@
##           master    #1180      +/-   ##
==========================================
+ Coverage   76.22%   76.35%   +0.12%     
==========================================
  Files          56       56              
  Lines        3151     3151              
  Branches      475      475              
==========================================
+ Hits         2402     2406       +4     
+ Misses        582      578       -4     
  Partials      167      167              
Impacted Files Coverage Δ
src/publickey.ts 95.65% <0.00%> (+5.79%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@steveluscher steveluscher merged commit 43e9be6 into solana-labs:master Feb 23, 2023
@steveluscher steveluscher deleted the update-rpc-websockets branch February 23, 2023 22:38
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.73.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Contributor

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web3.js] Infinite recursion caused by _updateSubscriptions
2 participants