Skip to content

Fix registry keyed limiter clear#3

Merged
puffball1567 merged 1 commit into
mainfrom
fix/registry-keyed-clear
Jul 1, 2026
Merged

Fix registry keyed limiter clear#3
puffball1567 merged 1 commit into
mainfrom
fix/registry-keyed-clear

Conversation

@puffball1567

Copy link
Copy Markdown
Owner

Summary

  • wire registry-created keyed fixed-window limiters to their underlying clear operation
  • add registry and C ABI regression coverage for clearing named keyed limiters

Context

This was found while exercising the Rust C ABI bindings: direct keyed fixed-window clear worked, stored limiter clear worked, but registry-created keyed fixed-window clear always returned false.

Verification

  • nim r --nimcache:/tmp/flowbrigade-registry-clear-registry-nimcache -p:src tests/test_ratelimit_registry.nim
  • nim r --nimcache:/tmp/flowbrigade-registry-clear-cabi-nimcache -p:src tests/test_c_abi.nim
  • nim r --nimcache:/tmp/flowbrigade-registry-clear-all-nimcache -p:src tests/all.nim
  • nimble check

@puffball1567
puffball1567 merged commit eb3d6f7 into main Jul 1, 2026
2 checks passed
@puffball1567
puffball1567 deleted the fix/registry-keyed-clear branch July 1, 2026 10:15
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