Skip to content

Fix: prevent the collision between y-websocket reconnectTimeout and polling interval - #7822

Merged
mejo- merged 1 commit into
mainfrom
fix/polling-interval-collision
Oct 23, 2025
Merged

Fix: prevent the collision between y-websocket reconnectTimeout and polling interval#7822
mejo- merged 1 commit into
mainfrom
fix/polling-interval-collision

Conversation

@benjaminfrueh

Copy link
Copy Markdown
Contributor

Minimal change to handle the timing more gracefully until we decide on a solution.

…y handling the timing more gracefully
Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>
@codecov

codecovBot commented Oct 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.71%. Comparing base (177bbb9) to head (564b4c9).
⚠️ Report is 76 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #7822 +/- ##
==========================================
+ Coverage 52.89% 59.71% +6.82% 
==========================================
Files 501 500 -1 Lines 43330 38379 -4951 Branches 1063 1062 -1 ==========================================
Hits 22918 22918 + Misses 20305 15355 -4950 + Partials 107 106 -1 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@juliusknorr

Copy link
Copy Markdown
Member

/backport! to stable32

@mejo-
mejo- merged commit 0e79387 into mainOct 23, 2025
62 of 64 checks passed
@mejo-
mejo- deleted the fix/polling-interval-collision branch October 23, 2025 17:46
@github-actions

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@benjaminfrueh@juliusknorr@mejo-