Quote all the things / potential fix for #14

Seji64 · Oct 5, 2023 · d2d9141 · d2d9141
1 parent 5a43e33
commit d2d9141
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/Services/LDAPService.cs b/src/Services/LDAPService.cs
@@ -217,7 +217,7 @@ private static async Task<string> DecryptLAPSPayload(byte[] value, LdapCredentia
             string pythonBin = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "python" : "python3";
 
             var pythonCmd = Cli.Wrap(pythonBin)
-                            .WithArguments($"\"{pythonDecryptScriptPath}\" --user {ldapCredential.UserName} --password {ldapCredential.Password} --data {Convert.ToBase64String(value)})")
+                            .WithArguments($"\"{pythonDecryptScriptPath}\" --user \"{ldapCredential.UserName}\" --password \"{ldapCredential.Password}\" --data \"{Convert.ToBase64String(value)}\"")
                             .WithStandardOutputPipe(PipeTarget.ToStringBuilder(pythonScriptResult));
 
             await pythonCmd.ExecuteAsync();