By default log when the rate limit is hit

matt-aitken · matt-aitken · commit a2993e36ba4c · 2024-05-28T13:41:52.000+01:00
diff --git a/apps/webapp/app/services/rateLimiter.server.ts b/apps/webapp/app/services/rateLimiter.server.ts
@@ -51,7 +51,8 @@ export class RateLimiter {
       });
     }
 
-    if (!success && this.options.logFailure) {
+    //log these by default
+    if (!success && this.options.logFailure !== false) {
       logger.info(`RateLimiter (${this.options.keyPrefix}): rate limit exceeded`, {
         limit,
         reset,

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,8 @@ export class RateLimiter {`
`51`	`51`	`});`
`52`	`52`	`}`
`53`	`53`
`54`		`- if (!success && this.options.logFailure) {`
	`54`	`+ //log these by default`
	`55`	`+ if (!success && this.options.logFailure !== false) {`
`55`	`56`	logger.info(`RateLimiter (${this.options.keyPrefix}): rate limit exceeded`, {
`56`	`57`	`limit,`
`57`	`58`	`reset,`