Merge pull request #7 from stashfiler/dev

TheFrankyJoe · web-flow · commit 15a489e83a6e · 2019-04-12T10:03:36.000-04:00
Security fix
diff --git a/docker-compose-wsl.py b/docker-compose-wsl.py
@@ -80,10 +80,7 @@ def wsl_drive_map(string_to_map):
     )
 ]
 
-# Run the docker-compose binary in bash on WSL with the appropriate argument(s)
-docker_compose_process = subprocess.Popen(docker_compose_cmd, shell=True)
-
 try:
-    docker_compose_process.wait()
+    subprocess.run(docker_compose_cmd)
 except KeyboardInterrupt:
-    docker_compose_process.terminate()
+    pass

Original file line number	Diff line number	Diff line change
`@@ -80,10 +80,7 @@ def wsl_drive_map(string_to_map):`
`80`	`80`	`)`
`81`	`81`	`]`
`82`	`82`
`83`		`-# Run the docker-compose binary in bash on WSL with the appropriate argument(s)`
`84`		`-docker_compose_process = subprocess.Popen(docker_compose_cmd, shell=True)`
`85`		`-`
`86`	`83`	`try:`
`87`		`- docker_compose_process.wait()`
	`84`	`+ subprocess.run(docker_compose_cmd)`
`88`	`85`	`except KeyboardInterrupt:`
`89`		`- docker_compose_process.terminate()`
	`86`	`+ pass`