NTR: too early debug message instead of warning (#932)

Co-authored-by: Vitalij Mik <[email protected]>

CHANGELOG.md

@@ -7,7 +7,6 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ## [Unreleased] 
 
 ### Added
-
 - Returns for Shopware Commercial plugins are now transferred to Mollie when the return status is set to "Done" and can be canceled with the "Cancelled" status. Please note that refunds cannot be canceled after two hours.
 - MB Way payment method is now available for Mollie Payments.
 - Multibanco payment method is now available for Mollie Payments.
@@ -17,6 +16,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 - The minimum supported Shopware version is now 6.4.5.0.
 - Added a new Monolog channel "mollie." You can now add custom handlers and assign them to the Mollie channel.
+- When a webhook from mollie is sent too early to the shop, a debug message is logged instead of a warning.
 
 ### Fixed
 

src/Controller/Api/Webhook/WebhookControllerBase.php

@@ -5,6 +5,7 @@
 use Kiener\MolliePayments\Components\Subscription\Exception\SubscriptionSkippedException;
 use Kiener\MolliePayments\Components\Subscription\SubscriptionManager;
 use Kiener\MolliePayments\Controller\Storefront\Webhook\NotificationFacade;
+use Kiener\MolliePayments\Exception\WebhookIsTooEarlyException;
 use Kiener\MolliePayments\Repository\OrderRepository;
 use Kiener\MolliePayments\Repository\OrderTransactionRepository;
 use Psr\Log\LoggerInterface;
@@ -13,6 +14,7 @@
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
 
 class WebhookControllerBase extends AbstractController
 {
@@ -74,6 +76,17 @@ public function webhookAction(string $swTransactionId, Request $request, Context
             return new JsonResponse([
                 'success' => true
             ]);
+        } catch (WebhookIsTooEarlyException $exception) {
+            $this->logger->debug('Webhook too early', [
+                ['message' => $exception->getMessage()]
+            ]);
+            return new JsonResponse(
+                [
+                    'success' => false,
+                    'error' => $exception->getMessage()
+                ],
+                Response::HTTP_TOO_EARLY
+            );
         } catch (\Throwable $ex) {
             $this->logger->error(
                 'Error in Mollie Webhook for Transaction ' . $swTransactionId,

src/Controller/Storefront/Webhook/NotificationFacade.php

@@ -132,6 +132,7 @@ public function __construct(MollieGatewayInterface $gatewayMollie, OrderStatusCo
     /**
      * @param string $swTransactionId
      * @param Context $context
+     * @throws WebhookIsTooEarlyException
      * @throws CustomerCouldNotBeFoundException
      * @return void
      */

src/Controller/Storefront/Webhook/WebhookControllerBase.php

@@ -5,6 +5,7 @@
 use Kiener\MolliePayments\Components\Subscription\Exception\SubscriptionSkippedException;
 use Kiener\MolliePayments\Components\Subscription\SubscriptionManager;
 use Kiener\MolliePayments\Controller\Storefront\AbstractStoreFrontController;
+use Kiener\MolliePayments\Exception\WebhookIsTooEarlyException;
 use Kiener\MolliePayments\Repository\OrderRepository;
 use Kiener\MolliePayments\Repository\OrderTransactionRepository;
 use Psr\Log\LoggerInterface;
@@ -70,6 +71,15 @@ public function onWebhookReceived(SalesChannelContext $context, string $swTransa
             $this->notificationFacade->onNotify($swTransactionId, $context->getContext());
 
             return new JsonResponse(['success' => true]);
+        } catch (WebhookIsTooEarlyException $exception) {
+            $this->logger->debug(
+                'Webhook too early',
+                [
+                    'error' => $exception->getMessage()
+                ]
+            );
+
+            return new JsonResponse(['success' => false, 'error' => $exception->getMessage()], $exception->getStatusCode());
         } catch (ShopwareHttpException $exception) {
             $this->logger->warning(
                 'Warning in Webhook for Transaction ' . $swTransactionId,