strapi-plugin-payone-provider 1.5.5 → 1.5.7

package/admin/src/pages/App/components/ApplePayButton.js

@@ -42,7 +42,7 @@ const ApplePayButton = ({
   const checkApplePayAvailability = async () => {
     try {
       console.log("[Apple Pay] Checking availability...");
-
+      
       // Check secure context using browser's native property
       // This is the most reliable way to check if we're in a secure context
       const isSecureContext = typeof window !== 'undefined' && window.isSecureContext;
@@ -57,7 +57,7 @@ const ApplePayButton = ({
       // - 127.0.0.1 (even on HTTP)
       // - file:// URLs
       const isSecure = isSecureContext || isLocalhost;
-
+      
       console.log("[Apple Pay] Secure context check:", {
         isSecureContext: isSecureContext,
         protocol: protocol,
@@ -78,12 +78,12 @@ const ApplePayButton = ({
           reason: !isSecureContext ? "window.isSecureContext is false" : "Unknown"
         });
       }
-
+      
       // First, check if Payment Request API is available
       // Payment Request API works on HTTP too, but Apple Pay JS API requires HTTPS
       if (typeof window === 'undefined' || !window.PaymentRequest) {
         console.log("[Apple Pay] Payment Request API not available");
-
+        
         // Fallback: Check Apple Pay JS API (for Safari, requires HTTPS)
         if (typeof window !== 'undefined' && window.ApplePaySession && isSecure) {
           try {
@@ -95,11 +95,11 @@ const ApplePayButton = ({
             return { available: false, method: null, error: 'insecure_context' };
           }
         }
-
+        
         if (!isSecure && typeof window !== 'undefined' && window.ApplePaySession) {
           console.warn("[Apple Pay] Apple Pay JS API requires HTTPS. Using Payment Request API fallback.");
         }
-
+        
         console.log("[Apple Pay] No Apple Pay support found");
         return { available: false, method: null };
       }
@@ -152,11 +152,11 @@ const ApplePayButton = ({
         try {
           const canPay = await testRequest.canMakePayment();
           console.log("[Apple Pay] canMakePayment result:", canPay);
-
+          
           if (canPay) {
             return { available: true, method: 'paymentRequest' };
           }
-
+          
           // If PaymentRequest says no, try Apple Pay JS API as fallback (only on HTTPS)
           if (typeof window !== 'undefined' && window.ApplePaySession && isSecure) {
             try {
@@ -172,17 +172,17 @@ const ApplePayButton = ({
               }
             }
           }
-
+          
           return { available: false, method: null };
         } catch (e) {
           console.error("[Apple Pay] Error checking canMakePayment:", e);
-
+          
           // If it's insecure context error, we can't use Apple Pay JS API
           if (e.message && e.message.includes('insecure')) {
             console.warn("[Apple Pay] Insecure context detected. Apple Pay requires HTTPS.");
             return { available: false, method: null, error: 'insecure_context' };
           }
-
+          
           // For other errors, try Apple Pay JS API as fallback (only on HTTPS)
           if (typeof window !== 'undefined' && window.ApplePaySession && isSecure) {
             try {
@@ -193,7 +193,7 @@ const ApplePayButton = ({
               return { available: false, method: null };
             }
           }
-
+          
           return { available: false, method: null };
         }
       }
@@ -204,7 +204,7 @@ const ApplePayButton = ({
       const hostnameFinal = typeof window !== 'undefined' ? window.location.hostname : '';
       const isLocalhostFinal = hostnameFinal === 'localhost' || hostnameFinal === '127.0.0.1';
       const isSecureFinal = isSecureContextFinal || isLocalhostFinal;
-
+      
       if (isSecureFinal) {
         console.log("[Apple Pay] canMakePayment not available, assuming support (secure context)");
         return { available: true, method: 'paymentRequest' };
@@ -219,20 +219,20 @@ const ApplePayButton = ({
       }
     } catch (error) {
       console.error("[Apple Pay] Error checking availability:", error);
-
+      
       // Check if it's insecure context error
       if (error.message && error.message.includes('insecure')) {
         console.warn("[Apple Pay] Insecure context - Apple Pay requires HTTPS");
         return { available: false, method: null, error: 'insecure_context' };
       }
-
+      
       // Fallback: Try Apple Pay JS API (only on HTTPS)
       // Re-check secure context
       const isSecureContextFallback = typeof window !== 'undefined' && window.isSecureContext;
       const hostnameFallback = typeof window !== 'undefined' ? window.location.hostname : '';
       const isLocalhostFallback = hostnameFallback === 'localhost' || hostnameFallback === '127.0.0.1';
       const isSecureFallback = isSecureContextFallback || isLocalhostFallback;
-
+      
       if (typeof window !== 'undefined' && window.ApplePaySession && isSecureFallback) {
         try {
           const canMakePayments = ApplePaySession.canMakePayments();
@@ -244,16 +244,16 @@ const ApplePayButton = ({
           }
         }
       }
-
+      
       return { available: false, method: null };
     }
   };
 
   useEffect(() => {
     const scriptUrl = "https://applepay.cdn-apple.com/jsapi/1.latest/apple-pay-sdk.js";
-
+    
     console.log("[Apple Pay] Loading Apple Pay SDK script...");
-
+    
     if (document.querySelector(`script[src="${scriptUrl}"]`)) {
       console.log("[Apple Pay] Script already loaded");
       // Script already loaded, check if it's ready
@@ -270,20 +270,20 @@ const ApplePayButton = ({
     script.src = scriptUrl;
     script.crossOrigin = "anonymous";
     script.async = true;
-
+    
     script.onload = () => {
       console.log("[Apple Pay] SDK script loaded successfully");
       setTimeout(() => {
         initializeButton();
       }, 500);
     };
-
+    
     script.onerror = (error) => {
       console.error("[Apple Pay] Failed to load SDK script:", error);
       setIsLoading(false);
       setIsAvailable(false);
       setErrorMessage("Failed to load Apple Pay SDK. Please check Content Security Policy settings.");
-
+      
       // Even if script fails, try to use Payment Request API
       console.log("[Apple Pay] Trying Payment Request API as fallback...");
       setTimeout(() => {
@@ -306,22 +306,22 @@ const ApplePayButton = ({
   const initializeButton = async () => {
     try {
       console.log("[Apple Pay] Initializing button...");
-
-      const isSecure = typeof window !== 'undefined' &&
-        (window.location.protocol === 'https:' ||
-          window.location.hostname === 'localhost' ||
-          window.location.hostname === '127.0.0.1');
-
+      
+      const isSecure = typeof window !== 'undefined' && 
+                      (window.location.protocol === 'https:' || 
+                       window.location.hostname === 'localhost' || 
+                       window.location.hostname === '127.0.0.1');
+      
       console.log("[Apple Pay] Secure context check:", {
         protocol: window.location?.protocol,
         hostname: window.location?.hostname,
         isSecure: isSecure
       });
-
-      const isLocalhost = typeof window !== 'undefined' &&
-        (window.location.hostname === 'localhost' ||
-          window.location.hostname === '127.0.0.1');
-
+      
+      const isLocalhost = typeof window !== 'undefined' && 
+                         (window.location.hostname === 'localhost' || 
+                          window.location.hostname === '127.0.0.1');
+      
       if (!isSecure && !isLocalhost && window.location?.protocol === 'http:') {
         const errorMsg = "Apple Pay requires HTTPS. Please access this page via HTTPS (https://yourdomain.com) instead of HTTP. Localhost (http://localhost) is allowed for development.";
         setErrorMessage(errorMsg);
@@ -331,7 +331,7 @@ const ApplePayButton = ({
         console.warn("[Apple Pay] Current URL:", window.location.href);
         return;
       }
-
+      
       // Log context information
       console.log("[Apple Pay] Context info:", {
         protocol: window.location?.protocol,
@@ -340,16 +340,16 @@ const ApplePayButton = ({
         isLocalhost: isLocalhost,
         fullUrl: window.location?.href
       });
-
+      
       // Check availability
       const availability = await checkApplePayAvailability();
       console.log("[Apple Pay] Availability check result:", availability);
-
+      
       setIsAvailable(availability.available);
-
+      
       if (!availability.available) {
         let errorMsg = "Apple Pay is not available on this device or browser.";
-
+        
         if (isLocalhost) {
           errorMsg = "Apple Pay is not available on localhost. Apple Pay requires a registered domain with HTTPS. " +
             "For testing, please use a production domain with HTTPS or test on a device with Safari (iOS/macOS). " +
@@ -366,7 +366,7 @@ const ApplePayButton = ({
         } else {
           errorMsg += " Please use Safari on iOS, macOS, or iPadOS, or a browser that supports Payment Request API (Chrome, Edge, Safari).";
         }
-
+        
         setErrorMessage(errorMsg);
         setIsLoading(false);
         console.warn("[Apple Pay] Not available:", errorMsg);
@@ -380,14 +380,14 @@ const ApplePayButton = ({
       console.error("[Apple Pay] Initialization error:", error);
       setIsLoading(false);
       setIsAvailable(false);
-
+      
       // Check for insecure context error
       if (error.message && error.message.includes('insecure')) {
         setErrorMessage("Apple Pay requires HTTPS. Please access this page via HTTPS (https://yourdomain.com) instead of HTTP. Localhost is allowed for development.");
       } else {
         setErrorMessage(error.message || "Failed to initialize Apple Pay");
       }
-
+      
       if (onError) {
         onError(error);
       }
@@ -406,11 +406,11 @@ const ApplePayButton = ({
     });
 
     // Check HTTPS requirement
-    const isSecure = typeof window !== 'undefined' &&
-      (window.location.protocol === 'https:' ||
-        window.location.hostname === 'localhost' ||
-        window.location.hostname === '127.0.0.1');
-
+    const isSecure = typeof window !== 'undefined' && 
+                    (window.location.protocol === 'https:' || 
+                     window.location.hostname === 'localhost' || 
+                     window.location.hostname === '127.0.0.1');
+    
     if (!isSecure && window.location?.protocol === 'http:') {
       const errorMsg = "Apple Pay requires HTTPS. Please access this page via HTTPS.";
       console.error("[Apple Pay]", errorMsg);
@@ -520,17 +520,24 @@ const ApplePayButton = ({
 
           // Complete with the merchant session promise
           // If the promise rejects, Payment Request API will close the dialog
+          // This is expected behavior - we cannot proceed without a valid merchant session
           event.complete(merchantSessionPromise);
         } catch (error) {
-          console.error("[Apple Pay] Merchant validation error:", error);
-          // Only call onError if it's defined
+          console.error("[Apple Pay] Merchant validation error:", {
+            message: error.message,
+            stack: error.stack,
+            response: error.response
+          });
+          
+          // Call onError to notify the user
           if (typeof onError === 'function') {
-            onError(error);
+            onError(new Error(`Apple Pay merchant validation failed: ${error.message}. Please check your Payone Apple Pay configuration in PMI (CONFIGURATION → PAYMENT PORTALS → [Your Portal] → Apple Pay).`));
           }
-          // Complete with a promise that resolves to empty object
-          // This allows the dialog to continue even if validation fails
-          // Apple Pay will show an error but won't close immediately
-          event.complete(Promise.resolve({}));
+          
+          // Complete with a rejected promise
+          // This will cause Apple Pay to close the dialog, which is expected behavior
+          // We cannot proceed without a valid merchant session from Payone
+          event.complete(Promise.reject(error));
         }
       };
 
@@ -610,11 +617,11 @@ const ApplePayButton = ({
 
       // Extract payment token
       const paymentToken = response.details?.paymentToken || response.details?.token;
-
+      
       if (!paymentToken) {
         console.error("[Apple Pay] Payment token is missing from response");
         try {
-          await response.complete("fail");
+        await response.complete("fail");
         } catch (completeError) {
           console.error("[Apple Pay] Error completing payment with fail:", completeError);
         }
@@ -656,11 +663,11 @@ const ApplePayButton = ({
           // The callback should set the token in state and return success immediately
           // It should NOT process the payment yet - that will happen when user clicks the button
           const callbackResult = onTokenReceived(tokenString, {
-            paymentToken: paymentToken,
-            billingContact: response.payerName || response.details?.billingContact,
-            shippingContact: response.shippingAddress || response.details?.shippingAddress,
-            shippingOption: response.shippingOption || response.details?.shippingOption
-          });
+          paymentToken: paymentToken,
+          billingContact: response.payerName || response.details?.billingContact,
+          shippingContact: response.shippingAddress || response.details?.shippingAddress,
+          shippingOption: response.shippingOption || response.details?.shippingOption
+        });
 
           // If callback returns a promise, wait for it to resolve or reject
           if (callbackResult && typeof callbackResult.then === 'function') {
@@ -772,12 +779,12 @@ const ApplePayButton = ({
         }
       });
 
-      console.log("[Apple Pay] Merchant session received from backend:", {
-        hasData: !!merchantSession.data,
-        merchantIdentifier: merchantSession.data?.merchantIdentifier
-      });
+        console.log("[Apple Pay] Merchant session received from backend:", {
+          hasData: !!merchantSession.data,
+          merchantIdentifier: merchantSession.data?.merchantIdentifier
+        });
 
-      return merchantSession.data || merchantSession;
+        return merchantSession.data || merchantSession;
     } catch (error) {
       console.error("[Apple Pay] Merchant validation error:", {
         message: error.message,
@@ -796,10 +803,10 @@ const ApplePayButton = ({
         console.error("[Apple Pay] Server error - Check server logs for details.");
       }
 
-      // If validation fails, return empty object
-      // This allows the dialog to continue, though Apple Pay may show an error
-      // The user can still proceed with payment if they want
-      return {};
+      // If validation fails, we cannot proceed
+      // Apple Pay requires a valid merchant session from Payone
+      console.error("[Apple Pay] Merchant validation failed - cannot proceed without valid session");
+      throw error;
     }
   };
 

package/admin/src/pages/App/components/ApplePayConfig.js

@@ -41,16 +41,16 @@ const ApplePayConfig = ({
       ...config,
       countryCode: value
     };
-
+    
     // Auto-update currency if current currency is not supported in new country
     const newSupportedCurrencies = getSupportedCurrenciesForCountry(value);
     if (!newSupportedCurrencies.find(c => c.code === currencyCode)) {
       newConfig.currencyCode = newSupportedCurrencies[0]?.code || "USD";
     }
-
+    
     // Auto-update networks based on country
     newConfig.supportedNetworks = getSupportedNetworksForCountry(value);
-
+    
     onConfigChange(newConfig);
   };
 
@@ -66,7 +66,7 @@ const ApplePayConfig = ({
     const newNetworks = currentNetworks.includes(networkCode)
       ? currentNetworks.filter(n => n !== networkCode)
       : [...currentNetworks, networkCode];
-
+    
     onConfigChange({
       ...config,
       supportedNetworks: newNetworks
@@ -78,7 +78,7 @@ const ApplePayConfig = ({
     const newCapabilities = currentCapabilities.includes(capabilityCode)
       ? currentCapabilities.filter(c => c !== capabilityCode)
       : [...currentCapabilities, capabilityCode];
-
+    
     onConfigChange({
       ...config,
       merchantCapabilities: newCapabilities
@@ -100,43 +100,43 @@ const ApplePayConfig = ({
         {/* Country and Currency */}
         <Flex gap={4} wrap="wrap">
           <Box style={{ flex: 1, minWidth: "300px" }}>
-            <Select
+          <Select
               label="Country Code"
-              name="countryCode"
-              value={countryCode}
-              onChange={handleCountryChange}
-              hint="Select the country where your business operates"
-              required
-            >
-              {APPLE_PAY_SUPPORTED_COUNTRIES.map(country => (
-                <Option key={country.code} value={country.code}>
-                  {country.name} ({country.code})
-                </Option>
-              ))}
-            </Select>
-          </Box>
+            name="countryCode"
+            value={countryCode}
+            onChange={handleCountryChange}
+            hint="Select the country where your business operates"
+            required
+          >
+            {APPLE_PAY_SUPPORTED_COUNTRIES.map(country => (
+              <Option key={country.code} value={country.code}>
+                {country.name} ({country.code})
+              </Option>
+            ))}
+          </Select>
+        </Box>
 
           <Box style={{ flex: 1, minWidth: "300px" }}>
-            <Select
+          <Select
               label="Currency Code"
-              name="currencyCode"
-              value={currencyCode}
-              onChange={handleCurrencyChange}
+            name="currencyCode"
+            value={currencyCode}
+            onChange={handleCurrencyChange}
               hint={`Supported currencies for ${countryCode}`}
-              required
-            >
-              {supportedCurrencies.map(currency => (
-                <Option key={currency.code} value={currency.code}>
-                  {currency.name} ({currency.code}) {currency.symbol}
-                </Option>
-              ))}
-            </Select>
-            {supportedCurrencies.length === 0 && (
-              <Typography variant="pi" textColor="danger600" style={{ marginTop: "4px" }}>
-                No supported currencies for this country. Please select a different country.
-              </Typography>
-            )}
-          </Box>
+            required
+          >
+            {supportedCurrencies.map(currency => (
+              <Option key={currency.code} value={currency.code}>
+                {currency.name} ({currency.code}) {currency.symbol}
+              </Option>
+            ))}
+          </Select>
+          {supportedCurrencies.length === 0 && (
+            <Typography variant="pi" textColor="danger600" style={{ marginTop: "4px" }}>
+              No supported currencies for this country. Please select a different country.
+            </Typography>
+          )}
+        </Box>
         </Flex>
 
         {/* Button Style and Type */}
@@ -186,23 +186,23 @@ const ApplePayConfig = ({
             {APPLE_PAY_SUPPORTED_NETWORKS.map(network => {
               const isSupported = supportedNetworksForCountry.includes(network.code);
               const isSelected = supportedNetworks?.includes(network.code);
-
+              
               return (
                 <Box key={network.code} style={{ flex: "0 0 calc(50% - 8px)", minWidth: "250px" }}>
-                  <Checkbox
-                    name={`network-${network.code}`}
-                    checked={isSelected}
-                    onChange={() => handleNetworkToggle(network.code)}
-                    disabled={!isSupported}
-                    hint={!isSupported ? `Not supported in ${countryCode}` : undefined}
-                  >
-                    {network.name} ({network.code})
-                    {!isSupported && (
-                      <Typography variant="sigma" textColor="neutral500" style={{ marginLeft: "8px" }}>
+                <Checkbox
+                  name={`network-${network.code}`}
+                  checked={isSelected}
+                  onChange={() => handleNetworkToggle(network.code)}
+                  disabled={!isSupported}
+                  hint={!isSupported ? `Not supported in ${countryCode}` : undefined}
+                >
+                  {network.name} ({network.code})
+                  {!isSupported && (
+                    <Typography variant="sigma" textColor="neutral500" style={{ marginLeft: "8px" }}>
                         (Not available)
-                      </Typography>
-                    )}
-                  </Checkbox>
+                    </Typography>
+                  )}
+                </Checkbox>
                 </Box>
               );
             })}
@@ -225,16 +225,16 @@ const ApplePayConfig = ({
           <Flex wrap="wrap" gap={4} style={{ marginTop: "12px" }}>
             {APPLE_PAY_MERCHANT_CAPABILITIES.map(capability => {
               const isSelected = merchantCapabilities?.includes(capability.code);
-
+              
               return (
                 <Box key={capability.code} style={{ flex: "0 0 calc(50% - 8px)", minWidth: "250px" }}>
-                  <Checkbox
-                    name={`capability-${capability.code}`}
-                    checked={isSelected}
-                    onChange={() => handleCapabilityToggle(capability.code)}
-                  >
-                    {capability.name} - {capability.description}
-                  </Checkbox>
+                <Checkbox
+                  name={`capability-${capability.code}`}
+                  checked={isSelected}
+                  onChange={() => handleCapabilityToggle(capability.code)}
+                >
+                  {capability.name} - {capability.description}
+                </Checkbox>
                 </Box>
               );
             })}
@@ -256,49 +256,49 @@ const ApplePayConfig = ({
           </Typography>
           <Flex wrap="wrap" gap={4} style={{ marginTop: "12px" }}>
             <Box style={{ flex: "0 0 calc(33.333% - 11px)", minWidth: "200px" }}>
-              <Checkbox
-                name="requestPayerName"
-                checked={requestPayerName}
-                onChange={(checked) => onConfigChange({ ...config, requestPayerName: checked })}
-              >
-                Request Payer Name
-              </Checkbox>
+            <Checkbox
+              name="requestPayerName"
+              checked={requestPayerName}
+              onChange={(checked) => onConfigChange({ ...config, requestPayerName: checked })}
+            >
+              Request Payer Name
+            </Checkbox>
             </Box>
             <Box style={{ flex: "0 0 calc(33.333% - 11px)", minWidth: "200px" }}>
-              <Checkbox
-                name="requestBillingAddress"
-                checked={requestBillingAddress}
-                onChange={(checked) => onConfigChange({ ...config, requestBillingAddress: checked })}
-              >
-                Request Billing Address
-              </Checkbox>
+            <Checkbox
+              name="requestBillingAddress"
+              checked={requestBillingAddress}
+              onChange={(checked) => onConfigChange({ ...config, requestBillingAddress: checked })}
+            >
+              Request Billing Address
+            </Checkbox>
             </Box>
             <Box style={{ flex: "0 0 calc(33.333% - 11px)", minWidth: "200px" }}>
-              <Checkbox
-                name="requestPayerEmail"
-                checked={requestPayerEmail}
-                onChange={(checked) => onConfigChange({ ...config, requestPayerEmail: checked })}
-              >
-                Request Payer Email
-              </Checkbox>
+            <Checkbox
+              name="requestPayerEmail"
+              checked={requestPayerEmail}
+              onChange={(checked) => onConfigChange({ ...config, requestPayerEmail: checked })}
+            >
+              Request Payer Email
+            </Checkbox>
             </Box>
             <Box style={{ flex: "0 0 calc(33.333% - 11px)", minWidth: "200px" }}>
-              <Checkbox
-                name="requestPayerPhone"
-                checked={requestPayerPhone}
-                onChange={(checked) => onConfigChange({ ...config, requestPayerPhone: checked })}
-              >
-                Request Payer Phone
-              </Checkbox>
+            <Checkbox
+              name="requestPayerPhone"
+              checked={requestPayerPhone}
+              onChange={(checked) => onConfigChange({ ...config, requestPayerPhone: checked })}
+            >
+              Request Payer Phone
+            </Checkbox>
             </Box>
             <Box style={{ flex: "0 0 calc(33.333% - 11px)", minWidth: "200px" }}>
-              <Checkbox
-                name="requestShipping"
-                checked={requestShipping}
-                onChange={(checked) => onConfigChange({ ...config, requestShipping: checked })}
-              >
-                Request Shipping Address
-              </Checkbox>
+            <Checkbox
+              name="requestShipping"
+              checked={requestShipping}
+              onChange={(checked) => onConfigChange({ ...config, requestShipping: checked })}
+            >
+              Request Shipping Address
+            </Checkbox>
             </Box>
           </Flex>
         </Box> */}
@@ -309,7 +309,7 @@ const ApplePayConfig = ({
             Merchant Identifier
           </Typography>
           <Typography variant="pi" textColor="neutral600">
-            {settings?.mid || settings?.portalid
+            {settings?.mid || settings?.portalid 
               ? `Using: ${settings.mid || settings.portalid}`
               : "Merchant identifier will be obtained from Payone after domain verification. See documentation for setup instructions."
             }

package/package.json

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

package/server/services/applePayService.js

@@ -126,6 +126,9 @@ const initializeApplePaySession = async (strapi, params) => {
 
     strapi.log.info("[Apple Pay] Session initialization response:", JSON.stringify(responseData, null, 2));
     strapi.log.info("[Apple Pay] Response status:", responseData.status || responseData.Status);
+    strapi.log.info("[Apple Pay] Response errorcode:", responseData.errorcode || responseData.ErrorCode || "none");
+    strapi.log.info("[Apple Pay] Response errormessage:", responseData.errormessage || responseData.ErrorMessage || responseData.errortxt || responseData.ErrorTxt || "none");
+    strapi.log.info("[Apple Pay] All response keys:", Object.keys(responseData));
 
     if (responseData.errorcode || responseData.ErrorCode) {
       strapi.log.warn("[Apple Pay] Response contains error:", {
@@ -205,10 +208,12 @@ const validateApplePayMerchant = async (strapi, params) => {
       strapi.log.error("[Apple Pay] Failed to initialize session with Payone:", {
         message: error.message,
         status: error.response?.status,
-        data: error.response?.data
+        data: error.response?.data,
+        stack: error.stack
       });
-      // Return empty object on error - Payment Request API will handle it
-      return {};
+      // DO NOT return empty object - throw error instead
+      // Empty object causes Apple Pay to close dialog without proper error message
+      throw new Error(`Failed to initialize Apple Pay session with Payone: ${error.message}. Please check your Payone configuration and ensure Apple Pay is properly set up in PMI.`);
     }
 
     strapi.log.info("[Apple Pay] Session initialization result:", {
@@ -250,10 +255,12 @@ const validateApplePayMerchant = async (strapi, params) => {
       strapi.log.info("[Apple Pay] Extracted Apple Pay session data:", {
         hasBase64Session: !!applePaySessionBase64,
         sessionLength: applePaySessionBase64?.length,
-        workorderid: sessionResponse.workorderid
+        workorderid: sessionResponse.workorderid,
+        allResponseKeys: Object.keys(sessionResponse),
+        responseSample: JSON.stringify(sessionResponse).substring(0, 500)
       });
 
-      if (applePaySessionBase64) {
+      if (applePaySessionBase64 && applePaySessionBase64.length > 0) {
         try {
           // Decode BASE64 merchant session
           const merchantSessionJson = Buffer.from(applePaySessionBase64, 'base64').toString('utf-8');
@@ -312,10 +319,22 @@ const validateApplePayMerchant = async (strapi, params) => {
           throw new Error(`Failed to decode Apple Pay merchant session: ${decodeError.message}`);
         }
       } else {
-        // If no session data in response, we need to create a valid merchant session
-        // According to Payone docs, merchant identifier should be obtained from PMI
-        // after domain verification and onboarding
-        strapi.log.warn("[Apple Pay] No Apple Pay session data in response, creating merchant session from settings");
+        // CRITICAL: If Payone doesn't return merchant session, we cannot proceed
+        // Apple Pay requires the merchant session to be validated by Apple's servers
+        // Payone should return add_paydata[applepay_payment_session] after successful initialization
+        strapi.log.error("[Apple Pay] CRITICAL: No Apple Pay session data in Payone response!");
+        strapi.log.error("[Apple Pay] This means merchant validation will fail. Possible causes:");
+        strapi.log.error("[Apple Pay] 1. Apple Pay not properly configured in Payone PMI");
+        strapi.log.error("[Apple Pay] 2. Domain not verified in Payone PMI");
+        strapi.log.error("[Apple Pay] 3. Merchant identifier not configured in Payone PMI");
+        strapi.log.error("[Apple Pay] 4. Apple Pay onboarding not completed in Payone PMI");
+        strapi.log.error("[Apple Pay] Response status:", responseStatus);
+        strapi.log.error("[Apple Pay] Full response keys:", Object.keys(sessionResponse));
+        strapi.log.error("[Apple Pay] Response sample:", JSON.stringify(sessionResponse).substring(0, 1000));
+
+        // DO NOT create a fallback session - it will fail validation
+        // Instead, throw an error so the frontend knows validation failed
+        throw new Error("Payone did not return Apple Pay merchant session. Please ensure Apple Pay is properly configured in Payone Merchant Interface (PMI): CONFIGURATION → PAYMENT PORTALS → [Your Portal] → Apple Pay configuration. The merchant session must come from Payone after successful Apple Pay onboarding.");
 
         // Get merchant identifier from settings
         // According to Payone docs, merchant identifier should be visible in PMI after onboarding
@@ -378,18 +397,43 @@ const validateApplePayMerchant = async (strapi, params) => {
       }
     }
 
-    // If initialization failed, return empty object
-    // Payment Request API will handle it
-    strapi.log.warn("[Apple Pay] Session initialization failed, returning empty object");
-    return {};
+    // If initialization failed, we cannot proceed
+    // Payment Request API requires a valid merchant session
+    strapi.log.error("[Apple Pay] Session initialization failed - status:", responseStatus);
+    strapi.log.error("[Apple Pay] Full Payone response:", JSON.stringify(sessionResponse, null, 2));
+    strapi.log.error("[Apple Pay] This means merchant validation will fail.");
+    strapi.log.error("[Apple Pay] Possible causes:");
+    strapi.log.error("[Apple Pay] 1. Payone returned ERROR status - check errorcode and errormessage in response");
+    strapi.log.error("[Apple Pay] 2. Apple Pay not configured in Payone PMI");
+    strapi.log.error("[Apple Pay] 3. Domain not verified in Payone PMI");
+    strapi.log.error("[Apple Pay] 4. Merchant identifier not configured correctly");
+    strapi.log.error("[Apple Pay] 5. Apple Pay onboarding not completed");
+    
+    // Extract error details from Payone response
+    const errorCode = sessionResponse.errorcode || sessionResponse.ErrorCode;
+    const errorMessage = sessionResponse.errormessage || sessionResponse.ErrorMessage || sessionResponse.errortxt || sessionResponse.ErrorTxt;
+    
+    if (errorCode || errorMessage) {
+      strapi.log.error("[Apple Pay] Payone error details:", {
+        errorCode: errorCode,
+        errorMessage: errorMessage
+      });
+      throw new Error(`Payone Apple Pay initialization failed: ${errorCode ? `Error ${errorCode}` : ''} ${errorMessage || 'Unknown error'}. Please check your Payone Apple Pay configuration in PMI (CONFIGURATION → PAYMENT PORTALS → [Your Portal] → Apple Pay).`);
+    } else {
+      throw new Error(`Apple Pay session initialization failed with status: ${responseStatus || 'UNKNOWN'}. Please check your Payone Apple Pay configuration in PMI (CONFIGURATION → PAYMENT PORTALS → [Your Portal] → Apple Pay).`);
+    }
   } catch (error) {
     strapi.log.error("[Apple Pay] Merchant validation error:", {
       message: error.message,
       stack: error.stack,
-      response: error.response?.data
+      response: error.response?.data,
+      status: error.response?.status
     });
-    // Return empty object on error - Payone will handle validation
-    return {};
+
+    // DO NOT return empty object - this causes Apple Pay to close the dialog
+    // Instead, re-throw the error so the frontend can handle it properly
+    // The error message will help the user understand what went wrong
+    throw error;
   }
 };