Backup_Repos/Readarr
1
0
Fork 0
mirror of https://github.com/Readarr/Readarr synced 2025-12-20 15:23:55 +01:00
Code Issues Projects 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 d5f36d0144
commit 50c0b0dbaa
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

@ -252,14 +252,6 @@ public override ValidationResult Test()
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())
{
try

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

@ -11,6 +11,7 @@ public class CustomScriptSettingsValidator : AbstractValidator<CustomScriptSetti
public CustomScriptSettingsValidator()
{
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");
}
}