Fixed: Executing powershell and python scripts directly in Connect->Custom Scripts

This commit is contained in:
Taloth Saldono 2019-07-02 20:50:32 +02:00
parent c71b4bde86
commit df8ca250aa
2 changed files with 58 additions and 0 deletions

View File

@ -77,6 +77,54 @@ namespace NzbDrone.Common.Test
Subject.Exists(DummyApp.DUMMY_PROCCESS_NAME).Should().BeFalse(); Subject.Exists(DummyApp.DUMMY_PROCCESS_NAME).Should().BeFalse();
} }
[Test]
public void Should_be_able_to_start_powershell()
{
WindowsOnly();
var tempDir = GetTempFilePath();
var tempScript = Path.Combine(tempDir, "myscript.ps1");
Directory.CreateDirectory(tempDir);
File.WriteAllText(tempScript, "Write-Output 'Hello There'\r\n");
try
{
var result = Subject.StartAndCapture(tempScript);
result.Standard.First().Content.Should().Be("Hello There");
}
catch (Win32Exception ex) when (ex.NativeErrorCode == 2)
{
Assert.Fail("No Powershell available?!?");
}
}
[Test]
public void Should_be_able_to_start_python()
{
WindowsOnly();
var tempDir = GetTempFilePath();
var tempScript = Path.Combine(tempDir, "myscript.py");
Directory.CreateDirectory(tempDir);
File.WriteAllText(tempScript, "print(\"Hello There\")\r\n");
try
{
var result = Subject.StartAndCapture(tempScript);
result.Standard.First().Content.Should().Be("Hello There");
}
catch (Win32Exception ex) when (ex.NativeErrorCode == 2)
{
Assert.Inconclusive("No Python available");
}
}
[Test] [Test]
public void kill_all_should_kill_all_process_with_name() public void kill_all_should_kill_all_process_with_name()

View File

@ -365,6 +365,16 @@ namespace NzbDrone.Common.Processes
return ("cmd.exe", $"/c {path} {args}"); return ("cmd.exe", $"/c {path} {args}");
} }
if (OsInfo.IsWindows && path.EndsWith(".ps1", StringComparison.InvariantCultureIgnoreCase))
{
return ("powershell.exe", $"-ExecutionPolicy Bypass -NoProfile -File {path} {args}");
}
if (OsInfo.IsWindows && path.EndsWith(".py", StringComparison.InvariantCultureIgnoreCase))
{
return ("python.exe", $"{path} {args}");
}
return (path, args); return (path, args);
} }
} }