handle errors from get request

asciisd · Dec 3, 2019 · 8628318 · 8628318
1 parent 072fded
commit 8628318
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 21 deletions.
diff --git a/routes/web.php b/routes/web.php
@@ -12,4 +12,4 @@
 use Illuminate\Support\Facades\Route;
 
 Route::post('/response', 'KnetController@handleKnet')->name('response');
-Route::post('/error', 'KnetController@error')->name('error');
+Route::get('/error', 'KnetController@error')->name('error');
diff --git a/src/Http/Controllers/KnetController.php b/src/Http/Controllers/KnetController.php
@@ -70,4 +70,19 @@ protected function missingMethod($parameters = [])
     {
         return new Response;
     }
+
+    public function error(Request $request)
+    {
+        KnetResponseReceived::dispatch($request->all());
+
+        $knetResponseHandler = new KnetResponseHandler();
+
+        // update transaction
+        KnetTransaction::findByTrackId($request->input('trackid'))
+            ->update($knetResponseHandler->toArray());
+
+        if ($knetResponseHandler->hasErrors()) {
+            throw new KnetException($knetResponseHandler->error());
+        }
+    }
 }
diff --git a/src/KnetResponseHandler.php b/src/KnetResponseHandler.php
@@ -24,6 +24,12 @@ public function __construct()
                 }
             }
         } else {
+            $this->result['error_text'] = request('ErrorText');
+            $this->result['error'] = request('Error');
+            $this->result['paymentid'] = request('paymentid');
+            $this->result['avr'] = request('avr');
+            $this->result['result'] = 'FAILED';
+
             $this->error = request('ErrorText');
             $this->error_code = request('Error');
         }

diff --git a/src/KnetResponseSignature.php b/src/KnetResponseSignature.php
@@ -6,28 +6,8 @@
 
 abstract class KnetResponseSignature
 {
-    const EXPECTED_REFERER = 'https://kpaytest.com.kw/kpg/paymentrouter.htm';
-
     public static function verifyHeader($payload, $header, $trackid)
     {
-        $referer = $header['referer'][0];
-
-        if (empty($referer)) {
-            throw SignatureVerificationException::factory(
-                "No referer found with expected scheme",
-                $payload,
-                $header
-            );
-        }
-
-        if ($referer != self::EXPECTED_REFERER) {
-            throw SignatureVerificationException::factory(
-                "No referer found matching the expected referer for payload",
-                $payload,
-                $header
-            );
-        }
-
         if (KnetTransaction::findByTrackId($trackid) == null) {
             throw SignatureVerificationException::factory(
                 "No trackid found matching the expected",