Mirrors/Radarr
1
0
Fork 0
mirror of https://github.com/Radarr/Radarr.git synced 2024-11-19 17:32:38 +01:00
Code Issues Releases Wiki Activity

Always validate Custom Script path

Browse Source 
(cherry picked from commit c922cc5dc617dd776d4523cbf62376821c5a4ad9)
This commit is contained in:
Mark McDowall 2023-11-19 00:17:32 -08:00 committed by Bogdan
parent 073d15160d
commit 263534717d
2 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/NzbDrone.Core/Notifications/CustomScript/CustomScript.cs
View File

@ -336,14 +336,6 @@ public override ValidationResult Test()
failures.Add(new NzbDroneValidationFailure("Path", "File does not exist")); failures.Add(new NzbDroneValidationFailure("Path", "File does not exist"));
} }
foreach (var systemFolder in SystemFolders.GetSystemFolders())
{
if (systemFolder.IsParentPath(Settings.Path))
{
failures.Add(new NzbDroneValidationFailure("Path", $"Must not be a descendant of '{systemFolder}'"));
}
}
if (failures.Empty()) if (failures.Empty())
{ {
try try

1
src/NzbDrone.Core/Notifications/CustomScript/CustomScriptSettings.cs
View File

@ -11,6 +11,7 @@ public class CustomScriptSettingsValidator : AbstractValidator<CustomScriptSetti
public CustomScriptSettingsValidator() public CustomScriptSettingsValidator()
{ {
RuleFor(c => c.Path).IsValidPath(); RuleFor(c => c.Path).IsValidPath();
RuleFor(c => c.Path).SetValidator(new SystemFolderValidator()).WithMessage("Must not be a descendant of '{systemFolder}'");
RuleFor(c => c.Arguments).Empty().WithMessage("Arguments are no longer supported for custom scripts"); RuleFor(c => c.Arguments).Empty().WithMessage("Arguments are no longer supported for custom scripts");
} }
} }