Archive-Team
/
sonarr-repo-only
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

New: Don't treat 400 responses from Notifiarr as errors 

Closes #5953
This commit is contained in:
Mark McDowall 2023-09-17 23:56:17 -07:00
parent 32e1ae2f64
commit 5eb420bbe1
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/NzbDrone.Core/Notifications/Notifiarr/NotifiarrProxy.cs
View File

@ -53,8 +53,10 @@ namespace NzbDrone.Core.Notifications.Notifiarr
_logger.Error("HTTP 401 - API key is invalid"); _logger.Error("HTTP 401 - API key is invalid");
throw new NotifiarrException("API key is invalid"); throw new NotifiarrException("API key is invalid");
case 400: case 400:
// 400 responses shouldn't be treated as an actual error because it's a misconfiguration
// between Sonarr and Notifiarr for a specific event, but shouldn't stop all events.
_logger.Error("HTTP 400 - Unable to send notification. Ensure Sonarr Integration is enabled & assigned a channel on Notifiarr"); _logger.Error("HTTP 400 - Unable to send notification. Ensure Sonarr Integration is enabled & assigned a channel on Notifiarr");
throw new NotifiarrException("Unable to send notification. Ensure Sonarr Integration is enabled & assigned a channel on Notifiarr"); break;
case 502: case 502:
case 503: case 503:
case 504: case 504: