Quick test: Avoid middleware change to fix CLI tests?

TryGhost · Jan 23, 2025 · 1722292 · 1722292
1 parent a6d81a7
commit 1722292
Showing 1 changed file with 16 additions and 5 deletions.
diff --git a/ghost/members-api/lib/members-api.js b/ghost/members-api/lib/members-api.js
@@ -335,12 +335,23 @@ module.exports = function MembersAPI({
         }
     };
 
+    const getMagicLinkMiddleware = () => {
+        if (captchaService.isEnabled()) {
+            return Router().use(
+                body.json(),
+                captchaService.getMiddleware(),
+                forwardError((req, res) => routerController.sendMagicLink(req, res))
+            );
+        } else {
+            return Router().use(
+                body.json(),
+                forwardError((req, res) => routerController.sendMagicLink(req, res))
+            );
+        }
+    };
+
     const middleware = {
-        sendMagicLink: Router().use(
-            body.json(),
-            captchaService.getMiddleware(),
-            forwardError((req, res) => routerController.sendMagicLink(req, res))
-        ),
+        sendMagicLink: getMagicLinkMiddleware(),
         createCheckoutSession: Router().use(
             body.json(),
             forwardError((req, res) => routerController.createCheckoutSession(req, res))