Add quotes to environment vars in DockerWrapper

bbimber · bbimber · commit 1d931dbf8671 · 2024-12-03T17:26:47.000-08:00
diff --git a/SequenceAnalysis/api-src/org/labkey/api/sequenceanalysis/run/DockerWrapper.java b/SequenceAnalysis/api-src/org/labkey/api/sequenceanalysis/run/DockerWrapper.java
@@ -146,7 +146,7 @@ public void executeWithDocker(List<String> containerArgs, File workDir, Pipeline
 
             for (String key : _environment.keySet())
             {
-                writer.println("\t-e " + key + "=" + _environment.get(key) + " \\");
+                writer.println("\t-e " + key + "='" + _environment.get(key) + "' \\");
             }
             writer.println("\t" + _containerName + " \\");
             writer.println("\t/bin/bash " + dockerBashScript.getPath());

Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ public void executeWithDocker(List<String> containerArgs, File workDir, Pipeline`
`146`	`146`
`147`	`147`	`for (String key : _environment.keySet())`
`148`	`148`	`{`
`149`		`- writer.println("\t-e " + key + "=" + _environment.get(key) + " \\");`
	`149`	`+ writer.println("\t-e " + key + "='" + _environment.get(key) + "' \\");`
`150`	`150`	`}`
`151`	`151`	`writer.println("\t" + _containerName + " \\");`
`152`	`152`	`writer.println("\t/bin/bash " + dockerBashScript.getPath());`