mirror of
https://github.com/Radarr/Radarr.git
synced 2024-11-04 10:02:40 +01:00
Add minimum length as const in ApiKeyValidationCheck
(cherry picked from commit b06269544cfa11015f3fb938fd8c2ef07d9cac4a) Closes #8618
This commit is contained in:
parent
9df06b80bf
commit
43ed8d0c2b
@ -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());
|
||||
|
@ -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",
|
||||
|
Loading…
Reference in New Issue
Block a user