From 43ed8d0c2bf60694111bed099005d24acae3fcd8 Mon Sep 17 00:00:00 2001 From: Bogdan Date: Sat, 13 May 2023 15:23:01 +0300 Subject: [PATCH] Add minimum length as const in ApiKeyValidationCheck (cherry picked from commit b06269544cfa11015f3fb938fd8c2ef07d9cac4a) Closes #8618 --- .../HealthCheck/Checks/ApiKeyValidationCheck.cs | 8 +++++--- src/NzbDrone.Core/Localization/Core/en.json | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/NzbDrone.Core/HealthCheck/Checks/ApiKeyValidationCheck.cs b/src/NzbDrone.Core/HealthCheck/Checks/ApiKeyValidationCheck.cs index 77e76d2a3..eb0e38014 100644 --- a/src/NzbDrone.Core/HealthCheck/Checks/ApiKeyValidationCheck.cs +++ b/src/NzbDrone.Core/HealthCheck/Checks/ApiKeyValidationCheck.cs @@ -10,6 +10,8 @@ namespace NzbDrone.Core.HealthCheck.Checks [CheckOn(typeof(ConfigSavedEvent))] public class ApiKeyValidationCheck : HealthCheckBase { + private const int MinimumLength = 20; + private readonly IConfigFileProvider _configFileProvider; private readonly Logger _logger; @@ -22,11 +24,11 @@ public ApiKeyValidationCheck(IConfigFileProvider configFileProvider, ILocalizati public override HealthCheck Check() { - if (_configFileProvider.ApiKey.Length < 20) + if (_configFileProvider.ApiKey.Length < MinimumLength) { - _logger.Warn("Please update your API key to be at least 20 characters long. You can do this via settings or the config file"); + _logger.Warn("Please update your API key to be at least {0} characters long. You can do this via settings or the config file", MinimumLength); - return new HealthCheck(GetType(), HealthCheckResult.Warning, _localizationService.GetLocalizedString("ApiKeyValidationHealthCheckMessage"), "#invalid-api-key"); + return new HealthCheck(GetType(), HealthCheckResult.Warning, string.Format(_localizationService.GetLocalizedString("ApiKeyValidationHealthCheckMessage"), MinimumLength), "#invalid-api-key"); } return new HealthCheck(GetType()); diff --git a/src/NzbDrone.Core/Localization/Core/en.json b/src/NzbDrone.Core/Localization/Core/en.json index 761166e45..5d8dd02cd 100644 --- a/src/NzbDrone.Core/Localization/Core/en.json +++ b/src/NzbDrone.Core/Localization/Core/en.json @@ -48,7 +48,7 @@ "Announced": "Announced", "AnnouncedMsg": "Movie is announced", "ApiKey": "API Key", - "ApiKeyValidationHealthCheckMessage": "Please update your API key to be at least 20 characters long. You can do this via settings or the config file", + "ApiKeyValidationHealthCheckMessage": "Please update your API key to be at least {0} characters long. You can do this via settings or the config file", "AppDataDirectory": "AppData directory", "AppDataLocationHealthCheckMessage": "Updating will not be possible to prevent deleting AppData on Update", "ApplicationURL": "Application URL",