joe/kami-parse-server
Public Access
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Rate limiting can fail when using Parse Server option rateLimit.redisUrl with clusters (#8632)

Browse Source
This commit is contained in:
Daniel
2024-03-24 12:17:21 +11:00
committed by GitHub
parent cfcb943ba5
commit c277739623
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/middlewares.js
View File

@@ -531,19 +531,17 @@ export const addRateLimit = (route, config, cloud) => {
const redisStore = { const redisStore = {
connectionPromise: Promise.resolve(), connectionPromise: Promise.resolve(),
store: null, store: null,
connected: false,
}; };
if (route.redisUrl) { if (route.redisUrl) {
const client = createClient({ const client = createClient({
url: route.redisUrl, url: route.redisUrl,
}); });
redisStore.connectionPromise = async () => { redisStore.connectionPromise = async () => {
if (redisStore.connected) { if (client.isOpen) {
return; return;
} }
try { try {
await client.connect(); await client.connect();
redisStore.connected = true;
} catch (e) { } catch (e) {
const log = config?.loggerController || defaultLogger; const log = config?.loggerController || defaultLogger;
log.error(`Could not connect to redisURL in rate limit: ${e}`); log.error(`Could not connect to redisURL in rate limit: ${e}`);