From 183ade98e3aa40d46eeea8dd3412fafe1469ac3f Mon Sep 17 00:00:00 2001 From: Simon Lecoq <22963968+lowlighter@users.noreply.github.com> Date: Thu, 14 Sep 2023 05:49:43 +0200 Subject: [PATCH] fix(app/web): accomodate express rate limit breaking change --- source/app/web/instance.mjs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/source/app/web/instance.mjs b/source/app/web/instance.mjs index 4b13c995..ea00c4e4 100644 --- a/source/app/web/instance.mjs +++ b/source/app/web/instance.mjs @@ -81,9 +81,12 @@ export default async function({sandbox = false} = {}) { //Rate limiter middleware if (ratelimiter) { app.set("trust proxy", 1) + const disabled = (ratelimiter.max === 0) + if (disabled) + delete ratelimiter.max middlewares.push(ratelimit({ skip(req, _res) { - return !!cache.get(req.params.login) + return (disabled) || (!!cache.get(req.params.login)) }, message: "Too many requests: retry later", headers: true,