Skip to content

#691: restore the counter even when it reached zero - #710

Open
morphqdd wants to merge 1 commit into
zerocracy:masterfrom
morphqdd:691
Open

#691: restore the counter even when it reached zero#710
morphqdd wants to merge 1 commit into
zerocracy:masterfrom
morphqdd:691

Conversation

@morphqdd

@morphqdd morphqdd commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

untrack_request gives back the credit when a request never reached GitHub, but
both branches tested positive? on the already-decremented counter, so at zero
the credit was lost. That is exactly the value off_quota? compares against.

The restore now mirrors the decrement: it happens whenever the counter is known.

Closes #691

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.

RateLimit#untrack_request cannot restore the counter at zero, the boundary it exists for

1 participant