Update error handler for better logging.

islamic-network · Apr 16, 2019 · cca6cba · cca6cba
1 parent b5746ad
commit cca6cba
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/AlAdhanApi/Handler/AlAdhanHandler.php b/src/AlAdhanApi/Handler/AlAdhanHandler.php
@@ -5,10 +5,11 @@
 use AlAdhanApi\Helper\Log;
 use Slim\Http\Request;
 use Slim\Http\Response;
+use Exception;
 
 class AlAdhanHandler
 {
-    public function __invoke(Request $request, Response $response, $exception = null) {
+    public function __invoke(Request $request, Response $response, Exception $exception) {
 
         if ($exception instanceof WafKeyMismatchException) {
             $r = [
@@ -21,15 +22,21 @@ public function __invoke(Request $request, Response $response, $exception = null
         };
 
         $r = [
-            'code' => 500,
+            'code' => $exception->getCode(),
             'status' => 'Internal Server Error',
             'data' => 'Something went wrong when the server tried to process this request. Sorry!'
         ];
 
         $log = new Log();
-        $log->error( $exception->getCode() . ' : ' . $exception->getMessage() . ' | ' . $exception->getTraceAsString());
-
-        return $response->withJson($r, 500);
+        $log->error('AlAdhan Exception Triggered.',
+            [
+                'code' => $exception->getCode(),
+                'message' => $exception->getMessage(),
+                'trace' => $exception->getTraceAsString()
+        ]
+        );
+
+        return $response->withJson($r, $exception->getCode());
     }
 
 }