strapi-plugin-payone-provider 1.3.0 → 1.4.1

package/admin/src/pages/App/components/icons/BankIcon.js

@@ -1,10 +1,10 @@
-import React from 'react';
-
-const BankIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 2l3.09 6.26L22 9.27l-5 4.87 1.18 6.88L12 17.77l-6.18 3.25L7 14.14 2 9.27l6.91-1.01L12 2z" fill={color} />
-    <path d="M12 6l-1.5 3L7 8.5l2.5 2.5L8 14l4-2.5L16 14l-1.5-3L18 8.5l-3.5.5L12 6z" fill={color} />
-  </svg>
-);
-
-export default BankIcon;
+import React from 'react';
+
+const BankIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 2l3.09 6.26L22 9.27l-5 4.87 1.18 6.88L12 17.77l-6.18 3.25L7 14.14 2 9.27l6.91-1.01L12 2z" fill={color} />
+    <path d="M12 6l-1.5 3L7 8.5l2.5 2.5L8 14l4-2.5L16 14l-1.5-3L18 8.5l-3.5.5L12 6z" fill={color} />
+  </svg>
+);
+
+export default BankIcon;

package/admin/src/pages/App/components/icons/ChevronDownIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const ChevronDownIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M7.41 8.59L12 13.17l4.59-4.58L18 10l-6 6-6-6 1.41-1.41z" fill={color} />
-  </svg>
-);
-
-export default ChevronDownIcon;
+import React from 'react';
+
+const ChevronDownIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M7.41 8.59L12 13.17l4.59-4.58L18 10l-6 6-6-6 1.41-1.41z" fill={color} />
+  </svg>
+);
+
+export default ChevronDownIcon;

package/admin/src/pages/App/components/icons/ChevronUpIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const ChevronUpIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M7.41 15.41L12 10.83l4.59 4.58L18 14l-6-6-6 6z" fill={color} />
-  </svg>
-);
-
-export default ChevronUpIcon;
+import React from 'react';
+
+const ChevronUpIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M7.41 15.41L12 10.83l4.59 4.58L18 14l-6-6-6 6z" fill={color} />
+  </svg>
+);
+
+export default ChevronUpIcon;

package/admin/src/pages/App/components/icons/CreditCardIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const CreditCardIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M20 4H4c-1.11 0-1.99.89-1.99 2L2 18c0 1.11.89 2 2 2h16c1.11 0 2-.89 2-2V6c0-1.11-.89-2-2-2zm0 14H4v-6h16v6zm0-10H4V6h16v2z" fill={color} />
-  </svg>
-);
-
-export default CreditCardIcon;
+import React from 'react';
+
+const CreditCardIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M20 4H4c-1.11 0-1.99.89-1.99 2L2 18c0 1.11.89 2 2 2h16c1.11 0 2-.89 2-2V6c0-1.11-.89-2-2-2zm0 14H4v-6h16v6zm0-10H4V6h16v2z" fill={color} />
+  </svg>
+);
+
+export default CreditCardIcon;

package/admin/src/pages/App/components/icons/ErrorIcon.js

@@ -1,10 +1,10 @@
-import React from 'react';
-
-const ErrorIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm-2 15l-5-5 1.41-1.41L10 14.17l7.59-7.59L19 8l-9 9z" fill={color} />
-    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm5 11H7v-2h10v2z" fill={color} />
-  </svg>
-);
-
-export default ErrorIcon;
+import React from 'react';
+
+const ErrorIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm-2 15l-5-5 1.41-1.41L10 14.17l7.59-7.59L19 8l-9 9z" fill={color} />
+    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm5 11H7v-2h10v2z" fill={color} />
+  </svg>
+);
+
+export default ErrorIcon;

package/admin/src/pages/App/components/icons/InfoIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const InfoIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm1 15h-2v-6h2v6zm0-8h-2V7h2v2z" fill={color} />
-  </svg>
-);
-
-export default InfoIcon;
+import React from 'react';
+
+const InfoIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm1 15h-2v-6h2v6zm0-8h-2V7h2v2z" fill={color} />
+  </svg>
+);
+
+export default InfoIcon;

package/admin/src/pages/App/components/icons/PaymentIcon.js

@@ -1,10 +1,10 @@
-import React from 'react';
-
-const PaymentIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M2 8h20v12H2V8zm2 2v8h16V10H4z" fill={color} />
-    <path d="M2 4h20v2H2V4z" fill={color} />
-  </svg>
-);
-
-export default PaymentIcon;
+import React from 'react';
+
+const PaymentIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M2 8h20v12H2V8zm2 2v8h16V10H4z" fill={color} />
+    <path d="M2 4h20v2H2V4z" fill={color} />
+  </svg>
+);
+
+export default PaymentIcon;

package/admin/src/pages/App/components/icons/PendingIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const PendingIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 2C6.5 2 2 6.5 2 12S6.5 22 12 22 22 17.5 22 12 17.5 2 12 2M12 20C7.59 20 4 16.41 4 12S7.59 4 12 4 20 7.59 20 12 16.41 20 12 20M12.5 7H11V13L16.25 16.15L17 14.92L12.5 12.25V7Z" fill={color} />
-  </svg>
-);
-
-export default PendingIcon;
+import React from 'react';
+
+const PendingIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 2C6.5 2 2 6.5 2 12S6.5 22 12 22 22 17.5 22 12 17.5 2 12 2M12 20C7.59 20 4 16.41 4 12S7.59 4 12 4 20 7.59 20 12 16.41 20 12 20M12.5 7H11V13L16.25 16.15L17 14.92L12.5 12.25V7Z" fill={color} />
+  </svg>
+);
+
+export default PendingIcon;

package/admin/src/pages/App/components/icons/PersonIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const PersonIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 12c2.21 0 4-1.79 4-4s-1.79-4-4-4-4 1.79-4 4 1.79 4 4 4zm0 2c-2.67 0-8 1.34-8 4v2h16v-2c0-2.66-5.33-4-8-4z" fill={color} />
-  </svg>
-);
-
-export default PersonIcon;
+import React from 'react';
+
+const PersonIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 12c2.21 0 4-1.79 4-4s-1.79-4-4-4-4 1.79-4 4 1.79 4 4 4zm0 2c-2.67 0-8 1.34-8 4v2h16v-2c0-2.66-5.33-4-8-4z" fill={color} />
+  </svg>
+);
+
+export default PersonIcon;

package/admin/src/pages/App/components/icons/SuccessIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const SuccessIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm-2 15l-5-5 1.41-1.41L10 14.17l7.59-7.59L19 8l-9 9z" fill={color} />
-  </svg>
-);
-
-export default SuccessIcon;
+import React from 'react';
+
+const SuccessIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm-2 15l-5-5 1.41-1.41L10 14.17l7.59-7.59L19 8l-9 9z" fill={color} />
+  </svg>
+);
+
+export default SuccessIcon;

package/admin/src/pages/App/components/icons/WalletIcon.js

@@ -1,9 +1,9 @@
-import React from 'react';
-
-const WalletIcon = ({ size = 16, color = 'currentColor' }) => (
-  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
-    <path d="M21 7.28V5c0-1.1-.9-2-2-2H5c-1.11 0-2 .9-2 2v14c0 1.1.89 2 2 2h14c1.1 0 2-.9 2-2v-2.28c.59-.35 1-.98 1-1.72V9c0-.74-.41-1.37-1-1.72zM20 9v6h-7V9h7zM5 19V5h14v2h-6c-1.1 0-2 .9-2 2v6c0 1.1.9 2 2 2h6v2H5z" fill={color} />
-  </svg>
-);
-
-export default WalletIcon;
+import React from 'react';
+
+const WalletIcon = ({ size = 16, color = 'currentColor' }) => (
+  <svg width={size} height={size} viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+    <path d="M21 7.28V5c0-1.1-.9-2-2-2H5c-1.11 0-2 .9-2 2v14c0 1.1.89 2 2 2h14c1.1 0 2-.9 2-2v-2.28c.59-.35 1-.98 1-1.72V9c0-.74-.41-1.37-1-1.72zM20 9v6h-7V9h7zM5 19V5h14v2h-6c-1.1 0-2 .9-2 2v6c0 1.1.9 2 2 2h6v2H5z" fill={color} />
+  </svg>
+);
+
+export default WalletIcon;

package/admin/src/pages/App/components/icons/index.js

@@ -1,11 +1,11 @@
-export { default as PaymentIcon } from './PaymentIcon';
-export { default as PersonIcon } from './PersonIcon';
-export { default as CreditCardIcon } from './CreditCardIcon';
-export { default as BankIcon } from './BankIcon';
-export { default as WalletIcon } from './WalletIcon';
-export { default as ErrorIcon } from './ErrorIcon';
-export { default as SuccessIcon } from './SuccessIcon';
-export { default as PendingIcon } from './PendingIcon';
-export { default as InfoIcon } from './InfoIcon';
-export { default as ChevronDownIcon } from './ChevronDownIcon';
-export { default as ChevronUpIcon } from './ChevronUpIcon';
+export { default as PaymentIcon } from './PaymentIcon';
+export { default as PersonIcon } from './PersonIcon';
+export { default as CreditCardIcon } from './CreditCardIcon';
+export { default as BankIcon } from './BankIcon';
+export { default as WalletIcon } from './WalletIcon';
+export { default as ErrorIcon } from './ErrorIcon';
+export { default as SuccessIcon } from './SuccessIcon';
+export { default as PendingIcon } from './PendingIcon';
+export { default as InfoIcon } from './InfoIcon';
+export { default as ChevronDownIcon } from './ChevronDownIcon';
+export { default as ChevronUpIcon } from './ChevronUpIcon';

package/admin/src/pages/hooks/usePaymentActions.js

@@ -116,9 +116,15 @@ const usePaymentActions = () => {
 
       if (needsRedirectUrls) {
         const baseUrl = window.location.origin;
-        baseParams.successurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/success`;
-        baseParams.errorurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/error`;
-        baseParams.backurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/back`;
+        // Detect current context (admin or content-ui) from pathname
+        const currentPath = window.location.pathname;
+        const isContentUI = currentPath.includes('/content-ui') || currentPath.includes('/content-manager');
+        const basePath = isContentUI ? '/content-ui' : '/admin';
+        const pluginPath = '/plugins/strapi-plugin-payone-provider/payment';
+
+        baseParams.successurl = `${baseUrl}${basePath}${pluginPath}/success`;
+        baseParams.errorurl = `${baseUrl}${basePath}${pluginPath}/error`;
+        baseParams.backurl = `${baseUrl}${basePath}${pluginPath}/back`;
       }
 
       const tokenToUse = tokenParam || googlePayToken;
@@ -252,9 +258,15 @@ const usePaymentActions = () => {
 
       if (needsRedirectUrls) {
         const baseUrl = window.location.origin;
-        baseParams.successurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/success`;
-        baseParams.errorurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/error`;
-        baseParams.backurl = `${baseUrl}/admin/plugins/strapi-plugin-payone-provider/payment/back`;
+        // Detect current context (admin or content-ui) from pathname
+        const currentPath = window.location.pathname;
+        const isContentUI = currentPath.includes('/content-ui') || currentPath.includes('/content-manager');
+        const basePath = isContentUI ? '/content-ui' : '/admin';
+        const pluginPath = '/plugins/strapi-plugin-payone-provider/payment';
+
+        baseParams.successurl = `${baseUrl}${basePath}${pluginPath}/success`;
+        baseParams.errorurl = `${baseUrl}${basePath}${pluginPath}/error`;
+        baseParams.backurl = `${baseUrl}${basePath}${pluginPath}/back`;
       }
 
       const tokenToUse = tokenParam || googlePayToken;

package/admin/src/pages/utils/formatTransactionData.js

@@ -1,16 +1,16 @@
-export const formatTransactionData = (data) => {
-  const formattedData = [];
-  if (!data || typeof data !== "object") return formattedData;
-
-  for (const [key, value] of Object.entries(data)) {
-    if (value !== null && value !== undefined) {
-      formattedData.push({
-        key:
-          key.charAt(0).toUpperCase() + key.slice(1).replace(/([A-Z])/g, " $1"),
-        value: typeof value === "object" ? JSON.stringify(value) : String(value)
-      });
-    }
-  }
-
-  return formattedData;
+export const formatTransactionData = (data) => {
+  const formattedData = [];
+  if (!data || typeof data !== "object") return formattedData;
+
+  for (const [key, value] of Object.entries(data)) {
+    if (value !== null && value !== undefined) {
+      formattedData.push({
+        key:
+          key.charAt(0).toUpperCase() + key.slice(1).replace(/([A-Z])/g, " $1"),
+        value: typeof value === "object" ? JSON.stringify(value) : String(value)
+      });
+    }
+  }
+
+  return formattedData;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-payone-provider",
-  "version": "1.3.0",
+  "version": "1.4.1",
   "description": "Strapi plugin for Payone payment gateway integration",
   "license": "MIT",
   "maintainers": [

package/server/bootstrap.js

@@ -23,4 +23,46 @@ module.exports = async ({ strapi }) => {
       }
     });
   }
+
+  // Register custom routes for 3DS redirects at root level
+  // These routes handle redirects from Payone after 3DS authentication
+  // They work with both /admin/ and /content-ui/ paths
+  const pluginName = "strapi-plugin-payone-provider";
+
+  // Get the controller
+  const getController = () => {
+    return strapi.plugin(pluginName).controller("payone");
+  };
+
+  // All routes use the same handler - it detects success/error/back from URL path
+  // Routes match the plugin's internal payment callback URLs
+  const routes = [
+    "/admin/plugins/strapi-plugin-payone-provider/payment/success",
+    "/admin/plugins/strapi-plugin-payone-provider/payment/error",
+    "/admin/plugins/strapi-plugin-payone-provider/payment/back",
+    "/content-ui/plugins/strapi-plugin-payone-provider/payment/success",
+    "/content-ui/plugins/strapi-plugin-payone-provider/payment/error",
+    "/content-ui/plugins/strapi-plugin-payone-provider/payment/back"
+  ];
+
+  // Register routes using Koa router
+  try {
+    const Router = require('@koa/router');
+    const router = new Router();
+
+    routes.forEach(route => {
+      router.get(route, async (ctx) => {
+        const controller = getController();
+        return await controller.handle3DSCallback(ctx);
+      });
+    });
+
+    // Add router to the server app
+    if (strapi.server.app && typeof strapi.server.app.use === 'function') {
+      strapi.server.app.use(router.routes());
+      strapi.server.app.use(router.allowedMethods());
+    }
+  } catch (error) {
+    strapi.log.warn('Could not register 3DS callback routes:', error.message);
+  }
 };

package/server/controllers/payone.js

@@ -43,12 +43,25 @@ module.exports = ({ strapi }) => ({
     }
   },
 
+  async getPublicSettings(ctx) {
+    try {
+      const settings = await getPayoneService(strapi).getSettings();
+      ctx.body = {
+        data: {
+          mid: settings?.mid || null,
+          mode: settings?.mode || null
+        }
+      };
+    } catch (error) {
+      handleError(ctx, error);
+    }
+  },
+
   async updateSettings(ctx) {
     try {
       const { body } = ctx.request;
       const currentSettings = await getPayoneService(strapi).getSettings();
 
-      // Preserve existing key if hidden or not provided
       if (body.key === "***HIDDEN***" || !body.key) {
         body.key = currentSettings?.key;
       }
@@ -124,13 +137,39 @@ module.exports = ({ strapi }) => ({
   /**
    * Handle 3D Secure callback from Payone
    * This endpoint receives the callback after customer completes 3DS authentication
+   * Works with both /admin/ and /content-ui/ paths
    */
   async handle3DSCallback(ctx) {
     try {
-      strapi.log.info("3DS callback received:", ctx.request.body);
+      const isGetRequest = ctx.request.method === "GET";
+      const currentPath = ctx.request.url;
+
+      let resultType = "callback";
+      if (currentPath.includes("/success")) {
+        resultType = "success";
+      } else if (currentPath.includes("/error")) {
+        resultType = "error";
+      } else if (currentPath.includes("/back")) {
+        resultType = "cancelled";
+      }
+
+      const callbackData = isGetRequest ? ctx.query : ctx.request.body;
+      strapi.log.info(`3DS ${resultType} received (${ctx.request.method}):`, callbackData);
+      const result = await getPayoneService(strapi).handle3DSCallback(callbackData, resultType);
 
-      const callbackData = ctx.request.body;
-      const result = await getPayoneService(strapi).handle3DSCallback(callbackData);
+      if (isGetRequest) {
+        const isContentUI = currentPath.includes('/content-ui');
+        const basePath = isContentUI ? '/content-ui' : '/admin';
+        const pluginPath = '/plugins/strapi-plugin-payone-provider';
+
+        const queryParams = new URLSearchParams();
+        queryParams.set('3ds', resultType);
+        if (result.txid) queryParams.set('txid', result.txid);
+        if (result.status) queryParams.set('status', result.status);
+
+        const redirectUrl = `${basePath}${pluginPath}?${queryParams.toString()}`;
+        return ctx.redirect(redirectUrl);
+      }
 
       ctx.body = { data: result };
     } catch (error) {

package/server/routes/index.js

@@ -82,6 +82,15 @@ module.exports = {
   "content-api": {
     type: "content-api",
     routes: [
+      {
+        method: "GET",
+        path: "/settings",
+        handler: "payone.getPublicSettings",
+        config: {
+          policies: ["plugin::strapi-plugin-payone-provider.is-auth"],
+          auth: false
+        }
+      },
       {
         method: "POST",
         path: "/preauthorization",

package/server/services/paymentService.js

@@ -219,36 +219,48 @@ const refund = async (strapi, params) => {
 /**
  * Handle 3D Secure callback from Payone
  * This processes the callback after customer completes 3DS authentication
+ * Note: Payone's redirect callback typically doesn't include transaction details -
+ * the URL path (success/error/back) indicates the result.
  * @param {Object} strapi - Strapi instance
- * @param {Object} callbackData - Callback data from Payone
+ * @param {Object} callbackData - Callback data from Payone (may be empty or minimal)
+ * @param {string} resultType - Result type from URL path: 'success', 'error', 'cancelled', or 'callback'
  * @returns {Promise<Object>} Processed callback result
  */
-const handle3DSCallback = async (strapi, callbackData) => {
+const handle3DSCallback = async (strapi, callbackData, resultType = 'callback') => {
   try {
-    const parsedData = parseResponse(callbackData, strapi.log);
+    // Parse any data that Payone might have sent
+    const parsedData = callbackData && Object.keys(callbackData).length > 0
+      ? parseResponse(callbackData, strapi.log)
+      : {};
 
-    // Extract transaction information
+    // Extract transaction information if available
     const txid = extractTxId(parsedData);
-    const status = parsedData.status || parsedData.Status || "unknown";
     const reference = parsedData.reference || parsedData.Reference || null;
 
-    // Log the callback transaction
-    await logTransaction(strapi, {
-      txid: txid || null,
-      reference: reference || null,
-      status: status,
-      request_type: "3ds_callback",
-      amount: parsedData.amount || null,
-      currency: parsedData.currency || "EUR",
-      raw_request: callbackData,
-      raw_response: parsedData,
-      error_code: parsedData.Error?.ErrorCode || null,
-      error_message: parsedData.Error?.ErrorMessage || null,
-      customer_message: parsedData.Error?.CustomerMessage || null
+    // Determine status from resultType (URL path) since Payone callback may not include status
+    let status;
+    if (resultType === 'success') {
+      status = 'APPROVED';
+    } else if (resultType === 'error') {
+      status = 'ERROR';
+    } else if (resultType === 'cancelled') {
+      status = 'CANCELLED';
+    } else {
+      // Fallback to parsed data if available
+      status = parsedData.status || parsedData.Status || 'PENDING';
+    }
+
+    // Log for debugging purposes only (not saved to transaction history)
+    strapi.log.info("3DS callback processed:", {
+      resultType,
+      status,
+      txid,
+      reference,
+      callbackData
     });
 
     return {
-      success: status.toUpperCase() === "APPROVED" || status.toUpperCase() === "REDIRECT",
+      success: resultType === 'success',
       status: status,
       txid: txid,
       reference: reference,

package/server/services/payone.js

@@ -50,7 +50,7 @@ module.exports = ({ strapi }) => ({
   },
 
   // 3D Secure callback handler
-  async handle3DSCallback(callbackData) {
-    return await paymentService.handle3DSCallback(strapi, callbackData);
+  async handle3DSCallback(callbackData, resultType) {
+    return await paymentService.handle3DSCallback(strapi, callbackData, resultType);
   }
 });