core: disable debug log level when the server restarts. resolves #8315 (#9665)

This commit is contained in:
Diego Heras 2020-09-27 01:19:46 +02:00 committed by GitHub
parent aef813b8e2
commit fa317b1329
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 4 deletions

View File

@ -78,7 +78,8 @@ namespace Jackett.Server.Controllers
var saveDir = config.blackholedir;
var updateDisabled = config.updatedisabled;
var preRelease = config.prerelease;
var logging = config.logging;
var enhancedLogging = config.logging;
var basePathOverride = config.basepathoverride;
if (basePathOverride != null)
{
@ -101,9 +102,6 @@ namespace Jackett.Server.Controllers
serverConfig.RuntimeSettings.BasePath = serverService.BasePath();
configService.SaveConfig(serverConfig);
Helper.SetLogLevel(logging ? LogLevel.Debug : LogLevel.Info);
serverConfig.RuntimeSettings.TracingEnabled = logging;
if (omdbApiKey != serverConfig.OmdbApiKey || omdbApiUrl != serverConfig.OmdbApiUrl)
{
serverConfig.OmdbApiKey = omdbApiKey;
@ -188,10 +186,17 @@ namespace Jackett.Server.Controllers
if (webHostRestartNeeded)
{
// we have to restore log level when the server restarts because we are not saving the state in the
// configuration. when the server restarts the UI is inconsistent with the active log level
// https://github.com/Jackett/Jackett/issues/8315
setEnhancedLogLevel(false);
Thread.Sleep(500);
logger.Info("Restarting webhost due to configuration change");
Helper.RestartWebHost();
}
else
setEnhancedLogLevel(enhancedLogging);
serverConfig.ConfigChanged();
@ -200,5 +205,11 @@ namespace Jackett.Server.Controllers
[HttpGet]
public List<CachedLog> Logs() => logCache.Logs;
private void setEnhancedLogLevel(bool enabled)
{
Helper.SetLogLevel(enabled ? LogLevel.Debug : LogLevel.Info);
serverConfig.RuntimeSettings.TracingEnabled = enabled;
}
}
}