update: improve error handling for developers

Author: bajoski34

src/Flutterwave.php

@@ -27,7 +27,7 @@ final class Flutterwave
 
     private \Psr\Log\LoggerInterface $logger;
 
-    const VERSION = "1.0.4";
+    const VERSION = "2.1.1";
 
     /**
      * Flutterwave constructor
@@ -116,4 +116,4 @@ private function loadConfig(): void
         ];
         // TODO: $this->config = config('flutterwave');
     }
-}
+}

src/Http/ConfirmRequest.php

@@ -57,10 +57,21 @@ public function messages(): array
      */
     public function failedValidation(\Illuminate\Contracts\Validation\Validator $validator)
     {
+        $logger = Log::channel('flutterwave');
+        $logger->error('Flutterwave Validation failed in ConfirmRequest:', $validator->errors()->toArray());
+        // parent::failedValidation($validator); // Let Laravel handle the rest of the failed validation.
+        $errors = (new ValidationException($validator))->errors();
+
         if (!app()->isProduction()) {
-            Log::error('Flutterwave Validation failed in ConfirmRequest:', $validator->errors()->toArray());
+            throw new InvalidArgument("Flutterwave Validation failed in ConfirmRequest", $errors);
         }
 
-        parent::failedValidation($validator); // Let Laravel handle the rest of the failed validation
+        throw new HttpResponseException(response()->json(
+            [
+                'error' => $errors,
+                'status_code' => JsonResponse::HTTP_UNPROCESSABLE_ENTITY,
+            ],
+            JsonResponse::HTTP_UNPROCESSABLE_ENTITY
+        ));
     }
 }

src/Http/PaymentRequest.php

@@ -6,6 +6,11 @@
 
 use Illuminate\Foundation\Http\FormRequest;
 use Illuminate\Support\Facades\Log;
+use Illuminate\Validation\ValidationException;
+use Illuminate\Http\Exceptions\HttpResponseException;
+use Illuminate\Http\Response;
+use Illuminate\Http\JsonResponse;
+use Flutterwave\Payments\Exception\InvalidArgument;
 
 class PaymentRequest extends FormRequest
 {
@@ -57,9 +62,20 @@ public function messages(): array
      */
     public function failedValidation(\Illuminate\Contracts\Validation\Validator $validator)
     {
+        $logger = Log::channel('flutterwave');
+        $logger->error('Flutterwave Validation failed in PaymentRequest:', $validator->errors()->toArray());
+        $errors = (new ValidationException($validator))->errors();
+
         if (!app()->isProduction()) {
-            Log::error('Flutterwave Validation failed in ConfirmRequest:', $validator->errors()->toArray());
+            throw new InvalidArgument("Flutterwave Validation failed in PaymentRequest", $errors);
         }
-        parent::failedValidation($validator);
+
+        throw new HttpResponseException(response()->json(
+            [
+                'error' => $errors,
+                'status_code' => JsonResponse::HTTP_UNPROCESSABLE_ENTITY,
+            ],
+            JsonResponse::HTTP_UNPROCESSABLE_ENTITY
+        ));
     }
 }

src/Services/Transactions.php

@@ -42,7 +42,7 @@ public function __construct(Api $api, array $config)
     }
 
     private function handleMissingSecretKey( $config ): void {
-        if( !isset( $config['secret_key'] )) {
+        if( !isset( $config['secret_key'] ) || empty( $config['secret_key'] ) ) {
             throw new InvalidArgument('The secret key is required. please add it to the .env file.');
         }
     }

src/resources/views/errors/invalid.blade.php

@@ -184,6 +184,18 @@
         <h1>Error: Something Went Wrong.</h1>
         <p>Oops, something went wrong while processing the payment with Flutterwave. Please check the error details below to help you debug.</p>
 
+        @if(isset($_GET['errors']) && is_array($_GET['errors']))
+            <div class="error-details">
+                <strong>Validation Errors:</strong>
+                <ul>
+                    @foreach($_GET['errors'] as $field => $messages)
+                        @foreach((array)$messages as $message)
+                            <li><strong>{{ $field }}:</strong> {{ $message }}</li>
+                        @endforeach
+                    @endforeach
+                </ul>
+            </div>
+        @else
         <div id="stackTraceText" class="error-details">
             <strong>Error Message:</strong>
             <p>{{ $_GET['message'] ?? 'No error message available.' }}</p>
@@ -198,7 +210,7 @@
             <pre>{{ $stackTrace ?? 'No stack trace available.' }}</pre>
             <button class="copy-btn copy-btn-stacktrace" onclick="copyText('stackTraceText')">Copy Stack Trace</button>
         </div>
-
+        @endif
         <!-- System Information Section -->
         <div id="systemInfoText" class="system-info">
             <h2>System Information</h2>