Fixed: (RadarrImport) Treat redirects as HTTP errors

src/NzbDrone.Core/ImportLists/Radarr/RadarrImport.cs

@@ -80,21 +80,23 @@ namespace NzbDrone.Core.ImportLists.Radarr
             // Return early if there is not an API key
             if (Settings.ApiKey.IsNullOrWhiteSpace())
             {
-                return new
-                {
-                    devices = new List<object>()
-                };
+                return new { options = new List<object>() };
             }
 
             Settings.Validate().Filter("ApiKey").ThrowOnError();
 
             if (action == "getProfiles")
             {
-                var devices = _radarrV3Proxy.GetProfiles(Settings);
+                var profiles = _radarrV3Proxy.GetProfiles(Settings);
+
+                if (profiles == null)
+                {
+                    return new { options = new List<object>() };
+                }
 
                 return new
                 {
-                    options = devices.OrderBy(d => d.Name, StringComparer.InvariantCultureIgnoreCase)
+                    options = profiles.OrderBy(d => d.Name, StringComparer.InvariantCultureIgnoreCase)
                         .Select(d => new
                         {
                             Value = d.Id,
@@ -105,11 +107,16 @@ namespace NzbDrone.Core.ImportLists.Radarr
 
             if (action == "getTags")
             {
-                var devices = _radarrV3Proxy.GetTags(Settings);
+                var tags = _radarrV3Proxy.GetTags(Settings);
+
+                if (tags == null)
+                {
+                    return new { options = new List<object>() };
+                }
 
                 return new
                 {
-                    options = devices.OrderBy(d => d.Label, StringComparer.InvariantCultureIgnoreCase)
+                    options = tags.OrderBy(d => d.Label, StringComparer.InvariantCultureIgnoreCase)
                         .Select(d => new
                         {
                             Value = d.Id,
@@ -120,13 +127,16 @@ namespace NzbDrone.Core.ImportLists.Radarr
 
             if (action == "getRootFolders")
             {
-                Settings.Validate().Filter("ApiKey").ThrowOnError();
+                var remoteRootFolders = _radarrV3Proxy.GetRootFolders(Settings);
 
-                var remoteRootfolders = _radarrV3Proxy.GetRootFolders(Settings);
+                if (remoteRootFolders == null)
+                {
+                    return new { options = new List<object>() };
+                }
 
                 return new
                 {
-                    options = remoteRootfolders.OrderBy(d => d.Path, StringComparer.InvariantCultureIgnoreCase)
+                    options = remoteRootFolders.OrderBy(d => d.Path, StringComparer.InvariantCultureIgnoreCase)
                         .Select(d => new
                         {
                             Value = d.Path,

src/NzbDrone.Core/ImportLists/Radarr/RadarrV3Proxy.cs

@@ -63,6 +63,12 @@ namespace NzbDrone.Core.ImportLists.Radarr
                     return new ValidationFailure("ApiKey", "API Key is invalid");
                 }
 
+                if (ex.Response.HasHttpRedirect)
+                {
+                    _logger.Error(ex, "Radarr returned redirect and is invalid");
+                    return new ValidationFailure("BaseUrl", "Radarr URL is invalid, are you missing a URL base?");
+                }
+
                 _logger.Error(ex, "Unable to connect to import list.");
                 return new ValidationFailure(string.Empty, $"Unable to connect to import list: {ex.Message}. Check the log surrounding this error for details.");
             }
@@ -84,11 +90,18 @@ namespace NzbDrone.Core.ImportLists.Radarr
 
             var baseUrl = settings.BaseUrl.TrimEnd('/');
 
-            var request = new HttpRequestBuilder(baseUrl).Resource(resource).Accept(HttpAccept.Json)
-                .SetHeader("X-Api-Key", settings.ApiKey).Build();
+            var request = new HttpRequestBuilder(baseUrl).Resource(resource)
+                .Accept(HttpAccept.Json)
+                .SetHeader("X-Api-Key", settings.ApiKey)
+                .Build();
 
             var response = _httpClient.Get(request);
 
+            if ((int)response.StatusCode > 299)
+            {
+                throw new HttpException(response);
+            }
+
             var results = JsonConvert.DeserializeObject<List<TResource>>(response.Content);
 
             return results;