Skip to content

fix(ci): rustfmt for media hub preauth slot guard - #163

Merged
AlexanderWagnerDev merged 2 commits into
mainfrom
cursor/ci-autofix-automation-c75e
Aug 13, 2026
Merged

fix(ci): rustfmt for media hub preauth slot guard#163
AlexanderWagnerDev merged 2 commits into
mainfrom
cursor/ci-autofix-automation-c75e

Conversation

@cursor

@cursor cursor Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes CI failure on PR #162 where cargo fmt --check rejected the if guard formatting in src/cluster/media/hub.rs.

Root cause

PR #162 introduced a long if condition with the opening brace on the same line; rustfmt expects the brace on its own line.

Fix

Apply rustfmt to try_acquire_global_preauth_media_slot() in src/cluster/media/hub.rs.

Related

Open in Web View Automation 

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

cursoragent and others added 2 commits August 13, 2026 02:15
Mirror control-plane preauth limits on the cluster media listener:
track half-open TLS/auth handshakes separately from authenticated
inflight sessions, cap per-source preauth connections, and only count
authenticated peers against MEDIA_CONN_INFLIGHT after auth succeeds.

Co-authored-by: Alexander Wagner <info@alexanderwagnerdev.com>
Co-authored-by: Alexander Wagner <info@alexanderwagnerdev.com>
@AlexanderWagnerDev
AlexanderWagnerDev marked this pull request as ready for review August 13, 2026 11:13
@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing

@AlexanderWagnerDev
AlexanderWagnerDev merged commit 4378e6f into main Aug 13, 2026
15 checks passed
@AlexanderWagnerDev
AlexanderWagnerDev deleted the cursor/ci-autofix-automation-c75e branch August 13, 2026 11:13
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.

2 participants