Skip to content

fix(egress): fail closed on poisoned rate state - #1280

Merged
yvgude merged 1 commit into
yvgude:mainfrom
cedric013:fix/1279-egress-rate-poison-fail-closed
Jul 27, 2026
Merged

fix(egress): fail closed on poisoned rate state#1280
yvgude merged 1 commit into
yvgude:mainfrom
cedric013:fix/1279-egress-rate-poison-fail-closed

Conversation

@cedric013

Copy link
Copy Markdown
Contributor

Summary

  • make the egress rate limiter fail closed when the process-local mutex is poisoned
  • add a regression test that poisons the limiter state and verifies no panic/bypass occurs

Tests

  • rustfmt --config-path rustfmt.toml --check src/core/egress.rs
  • git diff --check -- src/core/egress.rs
  • cargo test --lib egress::tests::poisoned_rate_state_fails_closed_without_panicking (attempted locally, cancelled after long compile; CI should exercise it)

Closes #1279

@yvgude
yvgude merged commit 61b0326 into yvgude:main Jul 27, 2026
25 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

egress rate limiter should fail closed when state lock is poisoned

2 participants