Fix knaken signature error (#140)

Co-authored-by: Ivascu Madalin <[email protected]>
buckaroo-it · Mar 14, 2024 · 04afec9 · 04afec9
1 parent d0186c8
commit 04afec9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/config/routes.yml b/config/routes.yml
@@ -2,7 +2,7 @@ buckaroo_refund:
     path: buckaroo3/refund
     methods: [POST]
     defaults:
-        _controller: 'buckaroo.refund_controller::refund'
+        _controller: 'Buckaroo\PrestaShop\Controllers\admin\AdminRefundController::refund'
 
 buckaroo_config_paymentMethod:
     path: buckaroo3/config/payment/methods

diff --git a/config/services.yml b/config/services.yml
@@ -8,7 +8,7 @@ services:
       - "@buckaroo.refund.request.response.handler"
     public: true
 
-  buckaroo.refund_controller:
+  Buckaroo\PrestaShop\Controllers\admin\AdminRefundController:
     class: Buckaroo\PrestaShop\Controllers\admin\AdminRefundController
     arguments:
       - "@buckaroo.refund.request.quantity_based_builder"

diff --git a/vendor/buckaroo/sdk/src/Handlers/Reply/HttpPost.php b/vendor/buckaroo/sdk/src/Handlers/Reply/HttpPost.php
@@ -61,12 +61,19 @@ public function validate(): bool
 
         //Combine the array keys with value
         $data = array_map(function ($value, $key) {
+            if ($key === 'brq_service_knaken_buyer_name') {
+                $key = 'brq_service_knaken_buyer name';
+            }
+
+            if ($key === 'brq_service_knaken_buyer_uuid') {
+                $key = 'brq_service_knaken_buyer uuid';
+            }
             return $key . '=' . html_entity_decode($value);
         }, $data, array_keys($data));
 
         $dataString = implode('', $data) . trim($this->config->secretKey());
 
-        return hash_equals(
+        return hash_equals( 
             sha1($dataString),
             trim($this->data['brq_signature'] ?? $this->data['BRQ_SIGNATURE'] ?? null)
         );