Skip to content

feat(rate-limit): Allow overwriting the rate limit - #56346

Merged
nickvergessen merged 1 commit into
masterfrom
feat/noid/allow-overwriting-rate-limit
Nov 12, 2025
Merged

feat(rate-limit): Allow overwriting the rate limit#56346
nickvergessen merged 1 commit into
masterfrom
feat/noid/allow-overwriting-rate-limit

Conversation

@nickvergessen

Copy link
Copy Markdown
Member
  • Allow admins to overwrite the rate-limit of a specific controller method

Checklist

@nickvergessennickvergessen added this to the Nextcloud 33 milestone Nov 10, 2025
@nickvergessennickvergessen self-assigned this Nov 10, 2025
@nickvergessen
nickvergessen requested a review from a team as a code ownerNovember 10, 2025 14:58
@nickvergessen
nickvergessen requested review from CarlSchwan, icewind1991, leftybournes and yemkareems and removed request for a teamNovember 10, 2025 14:58
@nickvergessen
nickvergessenforce-pushed the feat/noid/allow-overwriting-rate-limit branch from 7fef089 to f72b469CompareNovember 10, 2025 15:17
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable32

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

@kesselb

kesselb commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

Here's an example of how to use it:

 'ratelimit_overwrite' => [
'dashboard.dashboardapi.getwidgets' => [
'user' => ['limit' => 10, 'period' => 60], // use anon instead of user for guest users or fallback of logged in users
],
],

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the feat/noid/allow-overwriting-rate-limit branch from f72b469 to 2b9083aCompareNovember 12, 2025 08:00
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Here's an example of how to use it:

I added it to sample config

@nickvergessen
nickvergessen merged commit 49324bc into masterNov 12, 2025
238 of 266 checks passed
@nickvergessen
nickvergessen deleted the feat/noid/allow-overwriting-rate-limit branch November 12, 2025 09:09
@nextcloud-botnextcloud-bot mentioned this pull request Jan 6, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@kesselb@miaulalala