New: Configure log file size limit in UI

(cherry picked from commit 35baebaf7280749d5dfe5440e28b425e45a22d21)

Closes #10318
pull/10212/head
martylukyy 5 months ago committed by Bogdan
parent c0e76544ef
commit 14e3e1fa35

@ -157,6 +157,7 @@ class GeneralSettings extends Component {
/> />
<LoggingSettings <LoggingSettings
advancedSettings={advancedSettings}
settings={settings} settings={settings}
onInputChange={onInputChange} onInputChange={onInputChange}
/> />

@ -30,12 +30,14 @@ const logLevelOptions = [
function LoggingSettings(props) { function LoggingSettings(props) {
const { const {
advancedSettings,
settings, settings,
onInputChange onInputChange
} = props; } = props;
const { const {
logLevel logLevel,
logSizeLimit
} = settings; } = settings;
return ( return (
@ -52,11 +54,30 @@ function LoggingSettings(props) {
{...logLevel} {...logLevel}
/> />
</FormGroup> </FormGroup>
<FormGroup
advancedSettings={advancedSettings}
isAdvanced={true}
>
<FormLabel>{translate('LogSizeLimit')}</FormLabel>
<FormInputGroup
type={inputTypes.NUMBER}
name="logSizeLimit"
min={1}
max={10}
unit="MB"
helpText={translate('LogSizeLimitHelpText')}
onChange={onInputChange}
{...logSizeLimit}
/>
</FormGroup>
</FieldSet> </FieldSet>
); );
} }
LoggingSettings.propTypes = { LoggingSettings.propTypes = {
advancedSettings: PropTypes.bool.isRequired,
settings: PropTypes.object.isRequired, settings: PropTypes.object.isRequired,
onInputChange: PropTypes.func.isRequired onInputChange: PropTypes.func.isRequired
}; };

@ -841,6 +841,8 @@
"LogLevel": "Log Level", "LogLevel": "Log Level",
"LogLevelTraceHelpTextWarning": "Trace logging should only be enabled temporarily", "LogLevelTraceHelpTextWarning": "Trace logging should only be enabled temporarily",
"LogOnly": "Log Only", "LogOnly": "Log Only",
"LogSizeLimit": "Log Size Limit",
"LogSizeLimitHelpText": "Maximum log file size in MB before archiving. Default is 1MB.",
"Logging": "Logging", "Logging": "Logging",
"Logs": "Logs", "Logs": "Logs",
"LongDateFormat": "Long Date Format", "LongDateFormat": "Long Date Format",

@ -61,6 +61,8 @@ namespace Radarr.Api.V3.Config
.Must((resource, path) => IsValidSslCertificate(resource)).WithMessage("Invalid SSL certificate file or password") .Must((resource, path) => IsValidSslCertificate(resource)).WithMessage("Invalid SSL certificate file or password")
.When(c => c.EnableSsl); .When(c => c.EnableSsl);
SharedValidator.RuleFor(c => c.LogSizeLimit).InclusiveBetween(1, 10);
SharedValidator.RuleFor(c => c.Branch).NotEmpty().WithMessage("Branch name is required, 'master' is the default"); SharedValidator.RuleFor(c => c.Branch).NotEmpty().WithMessage("Branch name is required, 'master' is the default");
SharedValidator.RuleFor(c => c.UpdateScriptPath).IsValidPath().When(c => c.UpdateMechanism == UpdateMechanism.Script); SharedValidator.RuleFor(c => c.UpdateScriptPath).IsValidPath().When(c => c.UpdateMechanism == UpdateMechanism.Script);

@ -21,6 +21,7 @@ namespace Radarr.Api.V3.Config
public string Password { get; set; } public string Password { get; set; }
public string PasswordConfirmation { get; set; } public string PasswordConfirmation { get; set; }
public string LogLevel { get; set; } public string LogLevel { get; set; }
public int LogSizeLimit { get; set; }
public string ConsoleLogLevel { get; set; } public string ConsoleLogLevel { get; set; }
public string Branch { get; set; } public string Branch { get; set; }
public string ApiKey { get; set; } public string ApiKey { get; set; }
@ -65,6 +66,7 @@ namespace Radarr.Api.V3.Config
// Username // Username
// Password // Password
LogLevel = model.LogLevel, LogLevel = model.LogLevel,
LogSizeLimit = model.LogSizeLimit,
ConsoleLogLevel = model.ConsoleLogLevel, ConsoleLogLevel = model.ConsoleLogLevel,
Branch = model.Branch, Branch = model.Branch,
ApiKey = model.ApiKey, ApiKey = model.ApiKey,

Loading…
Cancel
Save