From bf17bd4754efc82f74a3518a8306b0152b83b9c8 Mon Sep 17 00:00:00 2001
From: Jamie Rees <Jamie.rees@moneybarn.com>
Date: Thu, 28 Jun 2018 14:01:09 +0100
Subject: [PATCH] Stop spamming errors when FanArt doesn't have the image

---
 src/Ombi.Api.FanartTv/FanartTvApi.cs             |  2 ++
 src/Ombi.Api/Api.cs                              | 16 +++++++++++++---
 src/Ombi.Api/Request.cs                          |  2 +-
 .../NotificationMessageCurlys.cs                 |  1 -
 4 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/src/Ombi.Api.FanartTv/FanartTvApi.cs b/src/Ombi.Api.FanartTv/FanartTvApi.cs
index bc819311c..55caef72c 100644
--- a/src/Ombi.Api.FanartTv/FanartTvApi.cs
+++ b/src/Ombi.Api.FanartTv/FanartTvApi.cs
@@ -21,6 +21,7 @@ namespace Ombi.Api.FanartTv
         {
             var request = new Request($"tv/{tvdbId}", Endpoint, HttpMethod.Get);
             request.AddHeader("api-key", token);
+            request.IgnoreErrors = true;
             try
             {
                 return await Api.Request<TvResult>(request);
@@ -36,6 +37,7 @@ namespace Ombi.Api.FanartTv
         {
             var request = new Request($"movies/{movieOrImdbId}", Endpoint, HttpMethod.Get);
             request.AddHeader("api-key", token);
+            request.IgnoreErrors = true;
 
             return await Api.Request<MovieResult>(request);
         }
diff --git a/src/Ombi.Api/Api.cs b/src/Ombi.Api/Api.cs
index 98fff5e0c..b0e7066a8 100644
--- a/src/Ombi.Api/Api.cs
+++ b/src/Ombi.Api/Api.cs
@@ -39,7 +39,11 @@ namespace Ombi.Api
 
                 if (!httpResponseMessage.IsSuccessStatusCode)
                 {
-                    LogError(request, httpResponseMessage);
+                    if (!request.IgnoreErrors)
+                    {
+                        LogError(request, httpResponseMessage);
+                    }
+
                     if (request.Retry)
                     {
 
@@ -94,7 +98,10 @@ namespace Ombi.Api
                 var httpResponseMessage = await _client.SendAsync(httpRequestMessage);
                 if (!httpResponseMessage.IsSuccessStatusCode)
                 {
-                    LogError(request, httpResponseMessage);
+                    if (!request.IgnoreErrors)
+                    {
+                        LogError(request, httpResponseMessage);
+                    }
                 }
                 // do something with the response
                 var data = httpResponseMessage.Content;
@@ -112,7 +119,10 @@ namespace Ombi.Api
                 var httpResponseMessage = await _client.SendAsync(httpRequestMessage);
                 if (!httpResponseMessage.IsSuccessStatusCode)
                 {
-                    LogError(request, httpResponseMessage);
+                    if (!request.IgnoreErrors)
+                    {
+                        LogError(request, httpResponseMessage);
+                    }
                 }
             }
         }
diff --git a/src/Ombi.Api/Request.cs b/src/Ombi.Api/Request.cs
index 89c3a7f2d..cfd284f54 100644
--- a/src/Ombi.Api/Request.cs
+++ b/src/Ombi.Api/Request.cs
@@ -25,7 +25,7 @@ namespace Ombi.Api
         public string Endpoint { get; }
         public string BaseUrl { get; }
         public HttpMethod HttpMethod { get; }
-
+        public bool IgnoreErrors { get; set; }
         public bool Retry { get; set; }
         public List<HttpStatusCode> StatusCodeToRetry { get; set; } = new List<HttpStatusCode>();
 
diff --git a/src/Ombi.Notifications/NotificationMessageCurlys.cs b/src/Ombi.Notifications/NotificationMessageCurlys.cs
index 748f2adb5..f42126156 100644
--- a/src/Ombi.Notifications/NotificationMessageCurlys.cs
+++ b/src/Ombi.Notifications/NotificationMessageCurlys.cs
@@ -13,7 +13,6 @@ namespace Ombi.Notifications
 {
     public class NotificationMessageCurlys
     {
-
         public void Setup(NotificationOptions opts, FullBaseRequest req, CustomizationSettings s)
         {
             LoadIssues(opts);