summaryrefslogtreecommitdiff
path: root/packages/backend/src/server/api
diff options
context:
space:
mode:
authorHazelnoot <acomputerdog@gmail.com>2024-12-08 11:58:57 -0500
committerHazelnoot <acomputerdog@gmail.com>2024-12-08 11:58:57 -0500
commit91c9b67cb08094fc00eab685c0f24b2668a9d8fc (patch)
treebe839bb46a0ddfa996de3a60d2a5184d72872878 /packages/backend/src/server/api
parentincrease rate limit for `/api/endpoint` based on real-world testing (diff)
downloadsharkey-91c9b67cb08094fc00eab685c0f24b2668a9d8fc.tar.gz
sharkey-91c9b67cb08094fc00eab685c0f24b2668a9d8fc.tar.bz2
sharkey-91c9b67cb08094fc00eab685c0f24b2668a9d8fc.zip
bypass rate limits when factor is 0
Diffstat (limited to 'packages/backend/src/server/api')
-rw-r--r--packages/backend/src/server/api/SkRateLimiterService.ts4
1 files changed, 2 insertions, 2 deletions
diff --git a/packages/backend/src/server/api/SkRateLimiterService.ts b/packages/backend/src/server/api/SkRateLimiterService.ts
index 027d05310b..d9abbfd406 100644
--- a/packages/backend/src/server/api/SkRateLimiterService.ts
+++ b/packages/backend/src/server/api/SkRateLimiterService.ts
@@ -35,7 +35,7 @@ export class SkRateLimiterService {
}
public async limit(limit: RateLimit, actor: string, factor = 1): Promise<LimitInfo> {
- if (this.disabled) {
+ if (this.disabled || factor === 0) {
return {
blocked: false,
remaining: Number.MAX_SAFE_INTEGER,
@@ -46,7 +46,7 @@ export class SkRateLimiterService {
};
}
- if (factor <= 0) {
+ if (factor < 0) {
throw new Error(`Rate limit factor is zero or negative: ${factor}`);
}