Skip to content

fix(nextcloud-talk): stabilize room timeout validation#111490

Merged
steipete merged 1 commit into
mainfrom
fix/nextcloud-room-info-timeout-test
Jul 19, 2026
Merged

fix(nextcloud-talk): stabilize room timeout validation#111490
steipete merged 1 commit into
mainfrom
fix/nextcloud-room-info-timeout-test

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

Fixes an issue where the Nextcloud Talk real-HTTP timeout proof could expire before the local test server received the request under loaded CI runners.

Why This Change Was Made

The test keeps its hanging-request timeout contract but uses a 500 ms bound, leaving scheduling and server-startup headroom while remaining far below the production timeout.

User Impact

No product behavior changes. Plugin prerelease validation no longer fails because a 50 ms test-only deadline elapsed before request dispatch.

Evidence

  • Failed release child: run 29695677729, received remained false after two attempts.
  • Focused real-HTTP timeout test: 10 consecutive passes.
  • Formatting and diff checks: green.
  • autoreview: no accepted or actionable findings.

@openclaw-barnacle openclaw-barnacle Bot added channel: nextcloud-talk Channel integration: nextcloud-talk size: XS maintainer Maintainer-authored PR labels Jul 19, 2026
@steipete
steipete merged commit d4c7549 into main Jul 19, 2026
107 of 113 checks passed
@steipete
steipete deleted the fix/nextcloud-room-info-timeout-test branch July 19, 2026 17:08
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

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

Labels

channel: nextcloud-talk Channel integration: nextcloud-talk maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant