fix: update laserstream dep to fix connection leak#1065
Conversation
Manual Deploy AvailableYou can trigger a manual deploy of this PR branch to testnet: Alternative: Comment
Comment updated automatically when the PR is synchronized. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can customize the tone of the review comments and chat replies.Configure the |
* master: fix: update laserstream dep to fix connection leak (#1065)
Summary
Update
helius-laserstreamdependency to a version that fixes the gRPC connection leak.The upstream fix ensures that when a
StreamHandleis dropped, the underlying gRPC connection is properlyaborted via a
watchchannel close signal, preventing leaked connections from accumulating.Details
The
helius-laserstreamgit dependency branch was changed fromv0.2.2-magicblocktov0.2.2-magicblock+conn-fix.The upstream fix (in laserstream-sdk) adds a
watch::Sender<bool>toStreamHandlewhoseDropimpl signals the background stream task to shut down gracefully. This prevents orphanedgRPC connections when the handle goes out of scope.
Summary by CodeRabbit