strapi-plugin-magic-link-v5 5.1.2 → 5.1.3

package/dist/server/index.js

@@ -222,35 +222,34 @@ Thanks.`,
     }
     return next();
   });
-  const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
-  setTimeout(() => {
+  setTimeout(async () => {
     try {
-      rateLimiter2.cleanupExpired();
+      const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
+      await rateLimiter2.cleanupExpired();
     } catch (error2) {
-      strapi2.log.debug("Rate limit cleanup skipped - DB not ready");
+      strapi2.log.debug("[CLEANUP] Rate limit cleanup skipped - DB not ready");
     }
   }, 1e4);
-  setInterval(() => {
+  setInterval(async () => {
     try {
-      rateLimiter2.cleanupExpired();
+      const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
+      await rateLimiter2.cleanupExpired();
     } catch (error2) {
-      strapi2.log.debug("Rate limit cleanup failed:", error2.message);
     }
   }, 30 * 60 * 1e3);
-  setTimeout(() => {
+  setTimeout(async () => {
     try {
       const otpService = strapi2.plugin("magic-link").service("otp");
-      otpService.cleanupExpiredCodes();
+      await otpService.cleanupExpiredCodes();
     } catch (error2) {
-      strapi2.log.debug("OTP cleanup skipped - DB not ready");
+      strapi2.log.debug("[CLEANUP] OTP cleanup skipped - DB not ready");
     }
   }, 1e4);
-  setInterval(() => {
+  setInterval(async () => {
     try {
       const otpService = strapi2.plugin("magic-link").service("otp");
-      otpService.cleanupExpiredCodes();
+      await otpService.cleanupExpiredCodes();
     } catch (error2) {
-      strapi2.log.error("OTP cleanup failed:", error2.message);
     }
   }, 5 * 60 * 1e3);
   try {
@@ -27365,13 +27364,11 @@ var rateLimiter$1 = ({ strapi: strapi2 }) => ({
   },
   /**
    * Clean up expired rate limit entries
+   * Uses the strapi instance from module closure (not global.strapi)
    */
   async cleanupExpired() {
     try {
-      if (!commonjsGlobal.strapi) {
-        throw new Error("Strapi instance not available");
-      }
-      const pluginStore = commonjsGlobal.strapi.store({
+      const pluginStore = strapi2.store({
         type: "plugin",
         name: "magic-link"
       });
@@ -27391,14 +27388,12 @@ var rateLimiter$1 = ({ strapi: strapi2 }) => ({
       });
       if (cleaned > 0) {
         await pluginStore.set({ key: "rate_limits", value: rateLimitData });
-        commonjsGlobal.strapi.log.info(`🧹 Cleaned up ${cleaned} expired rate limit entries`);
+        strapi2.log.info(`[CLEANUP] Cleaned up ${cleaned} expired rate limit entries`);
       }
       return { cleaned };
     } catch (error2) {
-      if (commonjsGlobal.strapi && commonjsGlobal.strapi.log) {
-        commonjsGlobal.strapi.log.error("Error cleaning up rate limits:", error2);
-      } else {
-        console.error("Error cleaning up rate limits:", error2);
+      if (strapi2 && strapi2.log) {
+        strapi2.log.debug("[CLEANUP] Rate limit cleanup skipped:", error2.message);
       }
       return { cleaned: 0 };
     }
@@ -27448,14 +27443,14 @@ const QRCode = require$$2__default$3.default;
 const emailHelpers = emailHelpers$3;
 const cryptoUtils = crypto_1;
 var otp$1 = ({ strapi: strapi2 }) => {
-  let strapiInstance = strapi2;
+  const strapiInstance = strapi2;
   const getStrapi = () => {
-    if (commonjsGlobal.strapi) {
-      return commonjsGlobal.strapi;
-    }
     if (strapiInstance) {
       return strapiInstance;
     }
+    if (commonjsGlobal.strapi) {
+      return commonjsGlobal.strapi;
+    }
     throw new Error("Strapi instance not available");
   };
   const log = {
@@ -27740,26 +27735,31 @@ If you didn't request this code, you can safely ignore this email.
     },
     /**
      * Clean up expired OTP codes
+     * Uses strapiInstance from module closure for reliability in bundled plugins
      */
     async cleanupExpiredCodes() {
       try {
-        const activeStrapi = getStrapi();
+        if (!strapiInstance) {
+          return;
+        }
         const now = /* @__PURE__ */ new Date();
-        const expiredCodes = await activeStrapi.documents("plugin::magic-link.otp-code").findMany({
+        const expiredCodes = await strapiInstance.documents("plugin::magic-link.otp-code").findMany({
           filters: {
             expiresAt: { $lt: now }
           }
         });
         for (const code of expiredCodes) {
-          await activeStrapi.documents("plugin::magic-link.otp-code").delete({
+          await strapiInstance.documents("plugin::magic-link.otp-code").delete({
             documentId: code.documentId
           });
         }
         if (expiredCodes.length > 0) {
-          activeStrapi.log.info(`Cleaned up ${expiredCodes.length} expired OTP codes`);
+          strapiInstance.log.info(`[CLEANUP] Cleaned up ${expiredCodes.length} expired OTP codes`);
         }
       } catch (error2) {
-        log.error("Error cleaning up expired OTP codes:", error2);
+        if (strapiInstance && strapiInstance.log) {
+          strapiInstance.log.debug("[CLEANUP] OTP cleanup skipped:", error2.message);
+        }
       }
     },
     /**

package/dist/server/index.mjs

@@ -188,35 +188,34 @@ Thanks.`,
     }
     return next();
   });
-  const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
-  setTimeout(() => {
+  setTimeout(async () => {
     try {
-      rateLimiter2.cleanupExpired();
+      const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
+      await rateLimiter2.cleanupExpired();
     } catch (error2) {
-      strapi2.log.debug("Rate limit cleanup skipped - DB not ready");
+      strapi2.log.debug("[CLEANUP] Rate limit cleanup skipped - DB not ready");
     }
   }, 1e4);
-  setInterval(() => {
+  setInterval(async () => {
     try {
-      rateLimiter2.cleanupExpired();
+      const rateLimiter2 = strapi2.plugin("magic-link").service("rate-limiter");
+      await rateLimiter2.cleanupExpired();
     } catch (error2) {
-      strapi2.log.debug("Rate limit cleanup failed:", error2.message);
     }
   }, 30 * 60 * 1e3);
-  setTimeout(() => {
+  setTimeout(async () => {
     try {
       const otpService = strapi2.plugin("magic-link").service("otp");
-      otpService.cleanupExpiredCodes();
+      await otpService.cleanupExpiredCodes();
     } catch (error2) {
-      strapi2.log.debug("OTP cleanup skipped - DB not ready");
+      strapi2.log.debug("[CLEANUP] OTP cleanup skipped - DB not ready");
     }
   }, 1e4);
-  setInterval(() => {
+  setInterval(async () => {
     try {
       const otpService = strapi2.plugin("magic-link").service("otp");
-      otpService.cleanupExpiredCodes();
+      await otpService.cleanupExpiredCodes();
     } catch (error2) {
-      strapi2.log.error("OTP cleanup failed:", error2.message);
     }
   }, 5 * 60 * 1e3);
   try {
@@ -27331,13 +27330,11 @@ var rateLimiter$1 = ({ strapi: strapi2 }) => ({
   },
   /**
    * Clean up expired rate limit entries
+   * Uses the strapi instance from module closure (not global.strapi)
    */
   async cleanupExpired() {
     try {
-      if (!commonjsGlobal.strapi) {
-        throw new Error("Strapi instance not available");
-      }
-      const pluginStore = commonjsGlobal.strapi.store({
+      const pluginStore = strapi2.store({
         type: "plugin",
         name: "magic-link"
       });
@@ -27357,14 +27354,12 @@ var rateLimiter$1 = ({ strapi: strapi2 }) => ({
       });
       if (cleaned > 0) {
         await pluginStore.set({ key: "rate_limits", value: rateLimitData });
-        commonjsGlobal.strapi.log.info(`🧹 Cleaned up ${cleaned} expired rate limit entries`);
+        strapi2.log.info(`[CLEANUP] Cleaned up ${cleaned} expired rate limit entries`);
       }
       return { cleaned };
     } catch (error2) {
-      if (commonjsGlobal.strapi && commonjsGlobal.strapi.log) {
-        commonjsGlobal.strapi.log.error("Error cleaning up rate limits:", error2);
-      } else {
-        console.error("Error cleaning up rate limits:", error2);
+      if (strapi2 && strapi2.log) {
+        strapi2.log.debug("[CLEANUP] Rate limit cleanup skipped:", error2.message);
       }
       return { cleaned: 0 };
     }
@@ -27414,14 +27409,14 @@ const QRCode = require$$2$4;
 const emailHelpers = emailHelpers$3;
 const cryptoUtils = crypto_1;
 var otp$1 = ({ strapi: strapi2 }) => {
-  let strapiInstance = strapi2;
+  const strapiInstance = strapi2;
   const getStrapi = () => {
-    if (commonjsGlobal.strapi) {
-      return commonjsGlobal.strapi;
-    }
     if (strapiInstance) {
       return strapiInstance;
     }
+    if (commonjsGlobal.strapi) {
+      return commonjsGlobal.strapi;
+    }
     throw new Error("Strapi instance not available");
   };
   const log = {
@@ -27706,26 +27701,31 @@ If you didn't request this code, you can safely ignore this email.
     },
     /**
      * Clean up expired OTP codes
+     * Uses strapiInstance from module closure for reliability in bundled plugins
      */
     async cleanupExpiredCodes() {
       try {
-        const activeStrapi = getStrapi();
+        if (!strapiInstance) {
+          return;
+        }
         const now = /* @__PURE__ */ new Date();
-        const expiredCodes = await activeStrapi.documents("plugin::magic-link.otp-code").findMany({
+        const expiredCodes = await strapiInstance.documents("plugin::magic-link.otp-code").findMany({
           filters: {
             expiresAt: { $lt: now }
           }
         });
         for (const code of expiredCodes) {
-          await activeStrapi.documents("plugin::magic-link.otp-code").delete({
+          await strapiInstance.documents("plugin::magic-link.otp-code").delete({
             documentId: code.documentId
           });
         }
         if (expiredCodes.length > 0) {
-          activeStrapi.log.info(`Cleaned up ${expiredCodes.length} expired OTP codes`);
+          strapiInstance.log.info(`[CLEANUP] Cleaned up ${expiredCodes.length} expired OTP codes`);
         }
       } catch (error2) {
-        log.error("Error cleaning up expired OTP codes:", error2);
+        if (strapiInstance && strapiInstance.log) {
+          strapiInstance.log.debug("[CLEANUP] OTP cleanup skipped:", error2.message);
+        }
       }
     },
     /**

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "5.1.2",
+  "version": "5.1.3",
   "keywords": [],
   "type": "commonjs",
   "exports": {