Don't log debug messages for API key validation

Closes #7934

frontend/src/RootFolder/RootFolderRow.tsx

@@ -83,7 +83,7 @@ function RootFolderRow(props: RootFolderRowProps) {
         isOpen={isDeleteModalOpen}
         kind={kinds.DANGER}
         title={translate('RemoveRootFolder')}
-        message={translate('RemoveRootFolderWithSeriesMessageText', { path })}
+        message={translate('RemoveRootFolderMessageText', { path })}
         confirmLabel={translate('Remove')}
         onConfirm={onConfirmDelete}
         onCancel={onDeleteModalClose}

frontend/src/Settings/Tags/Tag.tsx

@@ -55,13 +55,13 @@ function Tag({ id, label }: TagProps) {
   }, []);
 
   const handleConfirmDeleteTag = useCallback(() => {
-    dispatch(deleteTag({ id }));
-  }, [id, dispatch]);
-
-  const handleDeleteTagModalClose = useCallback(() => {
     setIsDeleteTagModalOpen(false);
   }, []);
 
+  const handleDeleteTagModalClose = useCallback(() => {
+    dispatch(deleteTag({ id }));
+  }, [id, dispatch]);
+
   return (
     <Card
       className={styles.tag}

src/NzbDrone.Common/Instrumentation/NzbDroneLogger.cs

@@ -215,6 +215,7 @@ namespace NzbDrone.Common.Instrumentation
                 c.ForLogger("Microsoft.*").WriteToNil(LogLevel.Warn);
                 c.ForLogger("Microsoft.Hosting.Lifetime*").WriteToNil(LogLevel.Info);
                 c.ForLogger("Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware").WriteToNil(LogLevel.Fatal);
+                c.ForLogger("Sonarr.Http.Authentication.ApiKeyAuthenticationHandler").WriteToNil(LogLevel.Info);
             });
         }
 

src/NzbDrone.Core/Configuration/ConfigFileProvider.cs

@@ -209,20 +209,9 @@ namespace NzbDrone.Core.Configuration
                     return AuthenticationType.Forms;
                 }
 
-                var value = Enum.TryParse<AuthenticationType>(_authOptions.Method, out var enumValue)
+                return Enum.TryParse<AuthenticationType>(_authOptions.Method, out var enumValue)
                     ? enumValue
                     : GetValueEnum("AuthenticationMethod", AuthenticationType.None);
-
-#pragma warning disable CS0618 // Type or member is obsolete
-                if (value == AuthenticationType.Basic)
-#pragma warning restore CS0618 // Type or member is obsolete
-                {
-                    SetValue("AuthenticationMethod", AuthenticationType.Forms);
-
-                    return AuthenticationType.Forms;
-                }
-
-                return value;
             }
         }
 
@@ -397,12 +386,6 @@ namespace NzbDrone.Core.Configuration
             {
                 SetValue("EnableSsl", false);
             }
-#pragma warning disable CS0618 // Type or member is obsolete
-            if (AuthenticationMethod == AuthenticationType.Basic)
-#pragma warning restore CS0618 // Type or member is obsolete
-            {
-                SetValue("AuthenticationMethod", AuthenticationType.Forms);
-            }
         }
 
         private void DeleteOldValues()

src/NzbDrone.Core/Localization/Core/en.json

@@ -1739,7 +1739,7 @@
   "RemoveQueueItemRemovalMethodHelpTextWarning": "'Remove from Download Client' will remove the download and the file(s) from the download client.",
   "RemoveQueueItemsRemovalMethodHelpTextWarning": "'Remove from Download Client' will remove the downloads and the files from the download client.",
   "RemoveRootFolder": "Remove Root Folder",
-  "RemoveRootFolderWithSeriesMessageText": "Are you sure you want to remove the root folder '{path}'? Files and folders will not be deleted from disk, and series in this root folder will not be removed from {appName}.",
+  "RemoveRootFolderMessageText": "Are you sure you want to remove the root folder '{path}'? Files and folders will not be deleted from disk, and series in this root folder will not be removed from {appName}.",
   "RemoveSelected": "Remove Selected",
   "RemoveSelectedBlocklistMessageText": "Are you sure you want to remove the selected items from the blocklist?",
   "RemoveSelectedItem": "Remove Selected Item",

src/NzbDrone.Host/Startup.cs

@@ -54,7 +54,7 @@ namespace NzbDrone.Host
                 b.ClearProviders();
                 b.SetMinimumLevel(LogLevel.Trace);
                 b.AddFilter("Microsoft.AspNetCore", LogLevel.Warning);
-                b.AddFilter("Sonarr.Http.Authentication", LogLevel.Information);
+                b.AddFilter("Sonarr.Http.Authentication.ApiKeyAuthenticationHandler", LogLevel.Information);
                 b.AddFilter("Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager", LogLevel.Error);
                 b.AddNLog();
             });

src/Sonarr.Http/Authentication/AuthenticationBuilderExtensions.cs

@@ -30,7 +30,7 @@ namespace Sonarr.Http.Authentication
 
         public static AuthenticationBuilder AddAppAuthentication(this IServiceCollection services)
         {
-            services.AddOptions<CookieAuthenticationOptions>(nameof(AuthenticationType.Forms))
+            services.AddOptions<CookieAuthenticationOptions>(AuthenticationType.Forms.ToString())
                 .Configure<IConfigFileProvider>((options, configFileProvider) =>
                 {
                     // Replace diacritics and replace non-word characters to ensure cookie name doesn't contain any valid URL characters not allowed in cookie names
@@ -47,9 +47,12 @@ namespace Sonarr.Http.Authentication
                 });
 
             return services.AddAuthentication()
-                .AddNone(nameof(AuthenticationType.None))
-                .AddExternal(nameof(AuthenticationType.External))
-                .AddCookie(nameof(AuthenticationType.Forms))
+                .AddNone(AuthenticationType.None.ToString())
+                .AddExternal(AuthenticationType.External.ToString())
+#pragma warning disable CS0618 // Type or member is obsolete
+                .AddCookie(AuthenticationType.Basic.ToString())
+#pragma warning restore CS0618 // Type or member is obsolete
+                .AddCookie(AuthenticationType.Forms.ToString())
                 .AddApiKey("API", options =>
                 {
                     options.HeaderName = "X-Api-Key";