Skip to content

test: order native socket setup and cancellation checks - #232

Merged
roodboi merged 2 commits into
nextfrom
fix/native-half-close-timeout-setup
Oct 9, 2026
Merged

roodboi merged 2 commits into
nextfrom
fix/native-half-close-timeout-setup

Conversation

@roodboi

@roodboi roodboi commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Two Rust socket tests configured or checked state before establishing the ordering they rely on. Configure the existing two-second read timeout before sending the half-close payload, and check dropped relay-command cancellation after the existing reactor liveness barrier. All payload, FIN, credential-refusal assertions and time budgets remain unchanged; production behavior is unchanged.

Validation: independent review of the exact two-file diff against current next; fresh default Rust suite (1,114 passed, 63 ignored, zero failures), both affected tests with all features, formatting, all-target default Clippy, and tracked-file privacy checks. Hosted CI remains required. The earlier hosted timeout-configuration EINVAL is recorded; its kernel cause is unproven.

Release intent: test-only ordering correction; no release or runtime behavior change.


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

@roodboi
roodboi merged commit 73ef475 into next Oct 9, 2026
13 checks passed
@roodboi
roodboi deleted the fix/native-half-close-timeout-setup branch October 9, 2026 02:06
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.

1 participant