add listener and failed notification

Author: svenvg93

app/Actions/Notifications/SendWebhookTestNotification.php

@@ -5,8 +5,11 @@
 use App\Models\Result;
 use App\Services\SpeedtestFakeResultGenerator;
 use Filament\Notifications\Notification;
+use Illuminate\Support\Facades\Event;
 use Illuminate\Support\Str;
 use Lorisleiva\Actions\Concerns\AsAction;
+use Spatie\WebhookServer\Events\WebhookCallFailedEvent;
+use Spatie\WebhookServer\Events\WebhookCallSucceededEvent;
 use Spatie\WebhookServer\WebhookCall;
 
 class SendWebhookTestNotification
@@ -27,9 +30,28 @@ public function handle(array $webhooks)
         // Generate a fake Result (NOT saved to database)
         $fakeResult = SpeedtestFakeResultGenerator::completed();
 
+        $hasFailure = false;
+
         foreach ($webhooks as $webhook) {
+            $url = $webhook['url'];
+            $succeeded = false;
+
+            // Listen for success/failure events for this specific webhook
+            Event::listen(WebhookCallSucceededEvent::class, function ($event) use ($url, &$succeeded) {
+                if ($event->webhookUrl === $url) {
+                    $succeeded = true;
+                }
+            });
+
+            Event::listen(WebhookCallFailedEvent::class, function ($event) use ($url, &$succeeded) {
+                if ($event->webhookUrl === $url) {
+                    $succeeded = false;
+                }
+            });
+
+            // Send webhook synchronously to get immediate result
             WebhookCall::create()
-                ->url($webhook['url'])
+                ->url($url)
                 ->payload([
                     'result_id' => Str::uuid(),
                     'site_name' => __('settings/notifications.test_notifications.webhook.payload'),
@@ -42,12 +64,25 @@ public function handle(array $webhooks)
                     'url' => url('/admin/results'),
                 ])
                 ->doNotSign()
-                ->dispatch();
+                ->dispatchSync();
+
+            if (! $succeeded) {
+                $hasFailure = true;
+            }
         }
 
-        Notification::make()
-            ->title(__('settings/notifications.test_notifications.webhook.sent'))
-            ->success()
-            ->send();
+        // Show appropriate notification based on results
+        if (! $hasFailure) {
+            Notification::make()
+                ->title(__('settings/notifications.test_notifications.webhook.sent'))
+                ->success()
+                ->send();
+        } else {
+            Notification::make()
+                ->title(__('settings/notifications.test_notifications.webhook.failed'))
+                ->body(__('settings/notifications.test_notifications.webhook.failed_body'))
+                ->danger()
+                ->send();
+        }
     }
 }

app/Listeners/LogWebhookFailure.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace App\Listeners;
+
+use Illuminate\Support\Facades\Log;
+use Spatie\WebhookServer\Events\WebhookCallFailedEvent;
+
+class LogWebhookFailure
+{
+    /**
+     * Handle the event.
+     */
+    public function handle(WebhookCallFailedEvent $event): void
+    {
+        Log::error('Webhook notification failed', [
+            'url' => $event->webhookUrl,
+            'attempt' => $event->attempt,
+            'http_verb' => $event->httpVerb,
+            'error_type' => $event->errorType,
+            'error_message' => $event->errorMessage,
+        ]);
+    }
+}

lang/en/settings/notifications.php

@@ -55,7 +55,9 @@
         ],
         'webhook' => [
             'add' => 'Add webhook URLs!',
-            'sent' => 'Test webhook notification sent.',
+            'sent' => 'Test webhook notification sent successfully.',
+            'failed' => 'Webhook notification failed.',
+            'failed_body' => 'Check the logs for details.',
             'payload' => 'Testing webhook notification',
         ],
     ],