Przeglądaj źródła

Merge pull request #24309 from overleaf/tm-recurly-webhook-ratelimit

Make a new less restrictive ratelimiter for the recurly callback

GitOrigin-RevId: 61bc39110a7ecc6e8f937478a9ccc965b555add5
Thomas Mees 1 rok temu
rodzic
commit
f5c92cb627

+ 6 - 1
services/web/app/src/Features/Subscription/SubscriptionRouter.mjs

@@ -168,7 +168,12 @@ export default {
     // recurly callback
     // recurly callback
     publicApiRouter.post(
     publicApiRouter.post(
       '/user/subscription/callback',
       '/user/subscription/callback',
-      RateLimiterMiddleware.rateLimit(subscriptionRateLimiter),
+      RateLimiterMiddleware.rateLimit(
+        new RateLimiter('recurly-callback', {
+          points: 200,
+          duration: 60,
+        })
+      ),
       AuthenticationController.requireBasicAuth({
       AuthenticationController.requireBasicAuth({
         [Settings.apis.recurly.webhookUser]: Settings.apis.recurly.webhookPass,
         [Settings.apis.recurly.webhookUser]: Settings.apis.recurly.webhookPass,
       }),
       }),