Thanks to visit codestin.com
Credit goes to github.com

Skip to content

wrangler: move ratelimit binding to stable #10541

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Sep 11, 2025
Merged

wrangler: move ratelimit binding to stable #10541

merged 3 commits into from
Sep 11, 2025

Conversation

qjex
Copy link
Contributor

@qjex qjex commented Sep 3, 2025 •
edited
Loading

Rate Limiting in Workers is now generally available. We can now remove ratelimit from unsafe bindings.

Old unsafe bindings should continue to work as before for now.

@qjex qjex requested a review from a team as a code owner September 3, 2025 09:04
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 3, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 528ef2e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
wrangler Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-pool-workers Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

penalosa
penalosa reviewed Sep 3, 2025
View reviewed changes
Copy link
Contributor

@penalosa penalosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should have a docs PR, and some tests. Probably a fixture test? I think there's an existing ratelimit one, so that could be extended

@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Sep 3, 2025
@pkg-pr-new pkg.pr.new
Copy link

pkg-pr-new bot commented Sep 3, 2025 •
edited
Loading

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@10541

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@10541

miniflare

npm i https://pkg.pr.new/miniflare@10541

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@10541

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@10541

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@10541

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@10541

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@10541

wrangler

npm i https://pkg.pr.new/wrangler@10541

commit: 528ef2e

@penalosa penalosa added the skip-v3-pr Skip validation of presence of a v3 backport PR label Sep 3, 2025
@qjex qjex force-pushed the ratelimit-ga branch 3 times, most recently from e8f6fe9 to 6805630 Compare September 3, 2025 13:00
@qjex
Copy link
Contributor Author

qjex commented Sep 3, 2025

Thanks @penalosa for the review. I added a fixture test, and linked the docs PR

@qjex qjex mentioned this pull request Sep 3, 2025
Merged
2 tasks
@qjex qjex requested a review from penalosa September 3, 2025 14:47
@GregBrimble GregBrimble mentioned this pull request Sep 4, 2025
Merged
6 tasks
@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Sep 8, 2025
@penalosa penalosa merged commit da24079 into cloudflare:main Sep 11, 2025
40 of 46 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Sep 11, 2025
@workers-devprod workers-devprod added the contribution [Holopin] Recognizes an open-source contribution, big or small label Sep 11, 2025
@holopin-bot Holopin Bot
Copy link

holopin-bot bot commented Sep 11, 2025

Congratulations @qjex, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cmfffdhuc0001l804cul9scwn

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

@workers-devprod workers-devprod mentioned this pull request Sep 11, 2025
Merged
petebacondarwin
petebacondarwin reviewed Sep 12, 2025
View reviewed changes
.changeset/five-drinks-stick.md

stable `ratelimit` binding

[Rate Limiting in Workers ](https://developers.cloudflare.com/workers/runtime-apis/bindings/rate-limit/) is now generally available, `ratelimit` can be removed from unsafe bindings.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The space here causes formatting checks to fail locally.
Not sure how that was not caught in CI?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#10630

emily-shen pushed a commit that referenced this pull request Sep 12, 2025
@qjex @emily-shen
wrangler: move ratelimit binding to stable (#10541)
494d367
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@petebacondarwin petebacondarwin petebacondarwin left review comments

@penalosa penalosa penalosa approved these changes

Assignees
No one assigned
Labels
contribution [Holopin] Recognizes an open-source contribution, big or small skip-v3-pr Skip validation of presence of a v3 backport PR
Projects
workers-sdk
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@qjex @petebacondarwin @penalosa @workers-devprod