Skip to content

ratelimit: reset on start#21086

Closed
icing wants to merge 1 commit intocurl:masterfrom
icing:ratelimit-reset-on-start
Closed

ratelimit: reset on start#21086
icing wants to merge 1 commit intocurl:masterfrom
icing:ratelimit-reset-on-start

Conversation

@icing
Copy link
Copy Markdown
Contributor

@icing icing commented Mar 24, 2026

On any Curl_rlimit_start() the rate limit needs to reset its values before calculating the effective step duration and adjust the tokens/burst per step.

Add two fields to the struct to remember the original values.

On any `Curl_rlimit_start()` the rate limit needs to reset its
values before calculating the effective step duration and adjust
the tokens/burst per step.

Add two fields to the struct to remember the original values.
@icing
Copy link
Copy Markdown
Contributor Author

icing commented Mar 24, 2026

@JimFuller-RedHat this should address your observed debug assertion.

@JimFuller-RedHat
Copy link
Copy Markdown
Contributor

LGTM

@icing icing requested a review from bagder March 24, 2026 13:11
@bagder bagder closed this in 797bc31 Mar 24, 2026
dkarpov1970 pushed a commit to dkarpov1970/curl that referenced this pull request Mar 25, 2026
On any `Curl_rlimit_start()` the rate limit needs to reset its
values before calculating the effective step duration and adjust
the tokens/burst per step.

Add two fields to the struct to remember the original values.

Closes curl#21086
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants