Fix: add try & catch on logger

Author: BeBlood

Controller/PaymentGatewayController.php

@@ -35,14 +35,16 @@ public function callbackAction(Request $request, EventDispatcherInterface $dispa
     {
         $data = $request->isMethod(Request::METHOD_POST) ? $request->request->all() : $request->query->all();
 
-        $this->logger->info(
-            sprintf(
-                '[gateway configuration alias: %s, data: %s, ip: %s]',
-                $configuration_alias,
-                json_encode($data),
-                json_encode($request->getClientIps())
-            )
-        );
+        try {
+            $this->logger->info(
+                sprintf(
+                    '[gateway configuration alias: %s, data: %s, ip: %s]',
+                    $configuration_alias,
+                    json_encode($data),
+                    json_encode($request->getClientIps())
+                )
+            );
+        } catch (\Exception $e) {}
 
         $paymentContext = $this
             ->paymentManager

Event/Subscriber/LoggerTransactionEventSubscriber.php

@@ -49,7 +49,7 @@ public function approve(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction approved: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction approved: '.$transactionEvent->getTransaction()->getId());
     }
 
     public function cancel(TransactionEvent $transactionEvent)
@@ -58,7 +58,7 @@ public function cancel(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction canceled: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction canceled: '.$transactionEvent->getTransaction()->getId());
     }
 
     public function create(TransactionEvent $transactionEvent)
@@ -67,7 +67,7 @@ public function create(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction created: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction created: '.$transactionEvent->getTransaction()->getId());
     }
 
     public function fail(TransactionEvent $transactionEvent)
@@ -76,7 +76,7 @@ public function fail(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction failed: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction failed: '.$transactionEvent->getTransaction()->getId());
     }
 
     public function pend(TransactionEvent $transactionEvent)
@@ -85,7 +85,7 @@ public function pend(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction pending: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction pending: '.$transactionEvent->getTransaction()->getId());
     }
 
     public function unverify(TransactionEvent $transactionEvent)
@@ -94,6 +94,12 @@ public function unverify(TransactionEvent $transactionEvent)
             return;
         }
 
-        $this->logger->info('Transaction unverified: '.$transactionEvent->getTransaction()->getId());
+        $this->log('Transaction unverified: '.$transactionEvent->getTransaction()->getId());
+    }
+
+    private function log(string $message) {
+        try {
+            $this->logger->info($message);
+        } catch (\Exception $e) {}
     }
 }

Payment/PaymentContext.php

@@ -88,9 +88,11 @@ public function handleGatewayCallback(Request $request): Transaction
         }
 
         if ($this->logger) {
-            $this->logger->info('Gateway response: ', [
-                'response' => json_encode($gatewayResponse->toArray()),
-            ]);
+            try {
+                $this->logger->info('Gateway response: ', [
+                    'response' => json_encode($gatewayResponse->toArray()),
+                ]);
+            } catch (\Exception $e) {}
         }
 
         return $transaction