From fd5ab27df615db6104dce8f0801513ae64ef7b64 Mon Sep 17 00:00:00 2001 From: Mark McDowall Date: Sun, 17 Sep 2023 23:56:17 -0700 Subject: [PATCH] New: Don't treat 400 responses from Notifiarr as errors (cherry picked from commit 5eb420bbe12f59d0a5392abf3d351be28ca210e6) Closes #2938 --- src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs b/src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs index 1de10cb5c..5b3e1979a 100644 --- a/src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs +++ b/src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs @@ -53,7 +53,10 @@ namespace NzbDrone.Core.Notifications.Notifiarr _logger.Error("HTTP 401 - API key is invalid"); throw new NotifiarrException("API key is invalid"); case 400: - throw new NotifiarrException("Unable to send notification. Ensure Readarr Integration is enabled & assigned a channel on Notifiarr"); + // 400 responses shouldn't be treated as an actual error because it's a misconfiguration + // between Readarr and Notifiarr for a specific event, but shouldn't stop all events. + _logger.Error("HTTP 400 - Unable to send notification. Ensure Readarr Integration is enabled & assigned a channel on Notifiarr"); + break; case 502: case 503: case 504: