Allow ssh/config to additionally accept a string

This is accepted by Net::SSH, research done by @jeremy in basecamp#908 (comment)

This is already documented as working correctly in https://github.com/basecamp/kamal/blob/74a06b0ccda616c86ebe1729d0795f39bcac9f00/lib/kamal/configuration/docs/ssh.yml#L65-L70

However, before this change only booleans were allowed because of the example configuration file.

lib/kamal/commands/base.rb

@@ -11,7 +11,7 @@ def initialize(config)
     end
 
     def run_over_ssh(*command, host:)
-      "ssh#{ssh_proxy_args} -t #{config.ssh.user}@#{host} -p #{config.ssh.port} '#{command.join(" ").gsub("'", "'\\\\''")}'"
+      "ssh#{ssh_proxy_args}#{ssh_config_args} -t #{config.ssh.user}@#{host} -p #{config.ssh.port} '#{command.join(" ").gsub("'", "'\\\\''")}'"
     end
 
     def container_id_for(container_name:, only_running: false)
@@ -94,5 +94,16 @@ def ssh_proxy_args
           " -o ProxyCommand='#{config.ssh.proxy.command_line_template}'"
         end
       end
+
+      def ssh_config_args
+        case config.ssh.config
+        when true, nil
+          ""
+        when false
+          " -F none"
+        when String
+          " -F #{Shellwords.escape(config.ssh.config)}"
+        end
+      end
   end
 end

lib/kamal/configuration/ssh.rb

@@ -7,7 +7,7 @@ class Kamal::Configuration::Ssh
 
   def initialize(config:)
     @ssh_config = config.raw_config.ssh || {}
-    validate! ssh_config
+    validate! ssh_config, with: Kamal::Configuration::Validator::Ssh
   end
 
   def user

lib/kamal/configuration/validator/ssh.rb

@@ -0,0 +1,19 @@
+class Kamal::Configuration::Validator::Ssh < Kamal::Configuration::Validator
+  SPECIAL_KEYS = [ "config" ]
+
+  def validate!
+    validate_against_example! \
+      config.except(*SPECIAL_KEYS),
+      example.except(*SPECIAL_KEYS)
+
+    validate_config_key! if config.key?("config")
+  end
+
+  private
+
+  def validate_config_key!
+    with_context(config["config"]) do
+      validate_type! config["config"], TrueClass, String
+    end
+  end
+end

test/commands/app_test.rb

@@ -300,6 +300,16 @@ class CommandsAppTest < ActiveSupport::TestCase
     assert_equal "ssh -t [email protected] -p 2222 'ls'", new_command.run_over_ssh("ls", host: "1.1.1.1")
   end
 
+  test "run over ssh with custom config" do
+    @config[:ssh] = { "config" => "config/ssh config" }
+    assert_equal "ssh -F config/ssh\\ config -t [email protected] -p 22 'ls'", new_command.run_over_ssh("ls", host: "1.1.1.1")
+  end
+
+  test "run over ssh with no config" do
+    @config[:ssh] = { "config" => false }
+    assert_equal "ssh -F none -t [email protected] -p 22 'ls'", new_command.run_over_ssh("ls", host: "1.1.1.1")
+  end
+
   test "run over ssh with proxy" do
     @config[:ssh] = { "proxy" => "2.2.2.2" }
     assert_equal "ssh -J [email protected] -t [email protected] -p 22 'ls'", new_command.run_over_ssh("ls", host: "1.1.1.1")

test/configuration/ssh_test.rb

@@ -42,4 +42,9 @@ class ConfigurationSshTest < ActiveSupport::TestCase
     config = Kamal::Configuration.new(@deploy.tap { |c| c.merge!(ssh: { "config" => false }) })
     assert_equal false, config.ssh.options[:config]
   end
+
+  test "ssh options with path to an ssh_config-file" do
+    config = Kamal::Configuration.new(@deploy.tap { |c| c.merge!(ssh: { "config" => "config/ssh_config" }) })
+    assert_equal "config/ssh_config", config.ssh.options[:config]
+  end
 end