Merge pull request #289 from lxShaDoWxl/master

fix: split video args
aerokube · Feb 19, 2021 · c0c040e · c0c040e
2 parents 103d482 + b3c4fb1
commit c0c040e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/selenoid/docker.go b/selenoid/docker.go
@@ -718,9 +718,11 @@ func (c *DockerConfigurator) Start() error {
 		cmd = append(cmd, "-conf", "/etc/selenoid/browsers.json")
 	}
 	if !contains(cmd, "-video-output-dir") && isVideoRecordingSupported(c.Logger, c.Version) {
-		cmd = append(cmd, "-video-output-dir", "/opt/selenoid/video/", "-video-recorder-image", videoRecorderImage)
+		cmd = append(cmd, "-video-output-dir", "/opt/selenoid/video/")
+	}
+	if !contains(cmd, "-video-recorder-image") && isVideoRecordingSupported(c.Logger, c.Version) {
+		cmd = append(cmd, "-video-recorder-image", videoRecorderImage)
 	}
-
 	if !c.DisableLogs && !contains(cmd, "-log-output-dir") && isLogSavingSupported(c.Logger, c.Version) {
 		cmd = append(cmd, "-log-output-dir", "/opt/selenoid/logs/")
 	}