From 755ff8783b9b4c1b4e5949fa417e0c85a8bc0e29 Mon Sep 17 00:00:00 2001 From: Hazelnoot Date: Wed, 11 Dec 2024 14:07:25 -0500 Subject: [PATCH] clarify naming of legacy rate limit methods --- packages/backend/src/server/api/SkRateLimiterService.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/backend/src/server/api/SkRateLimiterService.ts b/packages/backend/src/server/api/SkRateLimiterService.ts index d349e192e1..38c97b63df 100644 --- a/packages/backend/src/server/api/SkRateLimiterService.ts +++ b/packages/backend/src/server/api/SkRateLimiterService.ts @@ -55,15 +55,15 @@ export class SkRateLimiterService { private async limitLegacy(limit: Keyed, actor: string, factor: number): Promise { if (hasMaxLimit(limit)) { - return await this.limitMaxLegacy(limit, actor, factor); + return await this.limitLegacyMinMax(limit, actor, factor); } else if (hasMinLimit(limit)) { - return await this.limitMinLegacy(limit, actor, factor); + return await this.limitLegacyMinOnly(limit, actor, factor); } else { return disabledLimitInfo; } } - private async limitMaxLegacy(limit: Keyed, actor: string, factor: number): Promise { + private async limitLegacyMinMax(limit: Keyed, actor: string, factor: number): Promise { if (limit.duration === 0) return disabledLimitInfo; if (limit.duration < 0) throw new Error(`Invalid rate limit ${limit.key}: duration is negative (${limit.duration})`); if (limit.max < 1) throw new Error(`Invalid rate limit ${limit.key}: max is less than 1 (${limit.max})`); @@ -87,7 +87,7 @@ export class SkRateLimiterService { return await this.limitBucket(bucketLimit, actor, factor); } - private async limitMinLegacy(limit: Keyed, actor: string, factor: number): Promise { + private async limitLegacyMinOnly(limit: Keyed, actor: string, factor: number): Promise { if (limit.minInterval === 0) return disabledLimitInfo; if (limit.minInterval < 0) throw new Error(`Invalid rate limit ${limit.key}: minInterval is negative (${limit.minInterval})`);