Bug: Don't retry Apprise in case of a timeout (alexjustesen#2680)

Co-authored-by: Alex Justesen <[email protected]>

Author: svenvg93

app/Notifications/Apprise/SpeedtestNotification.php

@@ -10,6 +10,16 @@ class SpeedtestNotification extends Notification implements ShouldQueue
 {
     use Queueable;
 
+    /**
+     * The number of times the job may be attempted.
+     * Set to 1 to prevent duplicate notifications.
+     * Apprise may take >30s to respond (timeout), but still processes successfully.
+     * See #2653 and #2615
+     *
+     * @var int
+     */
+    public $tries = 1;
+
     public function __construct(
         public string $title,
         public string $body,

app/Notifications/Apprise/TestNotification.php

@@ -10,6 +10,16 @@ class TestNotification extends Notification implements ShouldQueue
 {
     use Queueable;
 
+    /**
+     * The number of times the job may be attempted.
+     * Set to 1 to prevent duplicate notifications.
+     * Apprise may take >30s to respond (timeout), but still processes successfully.
+     * See #2653 and #2615
+     *
+     * @var int
+     */
+    public $tries = 1;
+
     /**
      * Get the notification's delivery channels.
      *

app/Notifications/AppriseChannel.php

@@ -38,7 +38,7 @@ public function send(object $notifiable, Notification $notification): void
         ]);
 
         try {
-            $request = Http::timeout(30)
+            $request = Http::timeout(60)
                 ->withHeaders([
                     'Content-Type' => 'application/json',
                 ]);