Merge pull request #669 from handle_user_option_in_docker_transport

inspec · Mar 10, 2021 · 3dbb4f8 · 3dbb4f8
2 parents d5b3c9a + b6da849
commit 3dbb4f8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -79,6 +79,12 @@ train = Train.create('winrm',
 require 'train'
 train = Train.create('docker', host: 'container_id...')
 ```
+You can use `user` option to connect with privileged user on non root user images.
+
+```ruby
+require 'train'
+train = Train.create('docker', host: 'container_id...', user: 'root')
+```
 
 **AWS**
 

diff --git a/lib/train/transports/docker.rb b/lib/train/transports/docker.rb
@@ -88,7 +88,7 @@ def run_command_via_connection(cmd, &_data_handler)
       stdout, stderr, exit_status = @container.exec(
         [
           "/bin/sh", "-c", cmd
-        ]
+        ], user: @options[:user]
       )
       CommandResult.new(stdout.join, stderr.join, exit_status)
     rescue ::Docker::Error::DockerError => _