vagrant-scp-sync 0.5.15 → 0.5.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c3921e597aab855812426f77e3a91e807643551972b963e04992e04b43172c4
-  data.tar.gz: 1d3fd272ed5e7f3e5b111766a699a62e12ac593e61efcba981a4a4ad98779922
+  metadata.gz: 39eb1408a8d8e620bac881ec74ac17bab6834e2d0a5506278aa88922c0481585
+  data.tar.gz: 3239d37962771cc1a7e330a633617fce7f6fb96bbb4a522569389d10f9595beb
 SHA512:
-  metadata.gz: f1cab41bd06b2c0f9469a85b2d3926625c6bf15dd5a7ef7ba105ead7aa1e07e00545138d01af8b91683cbf78d7f13c33097de8f1df14f2fb541fc177fb75fb39
-  data.tar.gz: 5caa59840811a8bdeeabd325c9bb54b231e99568efde4c9cf39ae20f8d54136dcaf6383328cb83b14654129a9a31b43a506e084ec481544cbae92d8c86aa4a1e
+  metadata.gz: 1b0cf61de8a0475b78eb94db00cf064be8e135924ec6760c8e7b21330900fe6451c450854c2257051bf2d9db588b8976cfa56013c6196a01392d2f7e76de97bc
+  data.tar.gz: 2a295ad03d2082d32fbc5ac8dfea8753f44431c79315faaad0394f82f7d5b57856000ec59c817e8f02b5043da6bd61f0ceb4c3e41a92711f3c5989abffc86d5a

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## [0.5.16](https://github.com/STARTcloud/vagrant-scp-sync/compare/v0.5.15...v0.5.16) (2024-12-23)
+
+
+### Bug Fixes
+
+* single file sync - lint ([d953d76](https://github.com/STARTcloud/vagrant-scp-sync/commit/d953d761e37668f77d2b8d2c4417a3351bd260e8))
+* single file sync - lint ([9d179eb](https://github.com/STARTcloud/vagrant-scp-sync/commit/9d179eb36d01b85c75aade4a04a59eae8280264b))
+* single file sync - lint ([dad006f](https://github.com/STARTcloud/vagrant-scp-sync/commit/dad006fa9da87ac57a79a95124a741f9183a67f0))
+
 ## [0.5.15](https://github.com/STARTcloud/vagrant-scp-sync/compare/v0.5.14...v0.5.15) (2024-12-23)
 
 

data/lib/vagrant-scp-sync/action/scp_sync.rb

@@ -2,6 +2,7 @@
 
 require 'vagrant/util/platform'
 require 'vagrant/util/subprocess'
+require 'vagrant-scp-sync/ssh_options'
 
 module VagrantPlugins
   module ScpSync
@@ -17,7 +18,7 @@ module VagrantPlugins
         opts[:owner] ||= ssh_info[:username]
         opts[:group] ||= ssh_info[:username]
 
-        ssh_opts = build_ssh_options(ssh_info)
+        ssh_opts = SshOptions.build(ssh_info)
         scp_opts = build_scp_options(opts)
 
         delete = scp_opts.include?('--delete')
@@ -52,11 +53,13 @@ module VagrantPlugins
                      target_base
                    end
 
-          # Prepare target directory
-          parent_dir = File.dirname(target_files)
-          make_dir = build_ssh_command(ssh_opts, "sudo mkdir -p #{parent_dir}", ssh_info)
-          change_ownership = build_ssh_command(ssh_opts, "sudo chown -R #{opts[:owner]}:#{opts[:group]} #{parent_dir}", ssh_info)
-          change_permissions = build_ssh_command(ssh_opts, "sudo chmod 777 #{parent_dir}", ssh_info)
+          # Prepare remote target directory with proper permissions
+          target_dir = target_files
+          target_dir = File.dirname(target_files) unless target_is_dir || has_trailing_slash_target
+
+          make_dir = build_ssh_command(ssh_opts, "sudo mkdir -p #{target_dir}", ssh_info)
+          change_ownership = build_ssh_command(ssh_opts, "sudo chown -R #{opts[:owner]}:#{opts[:group]} #{target_dir}", ssh_info)
+          change_permissions = build_ssh_command(ssh_opts, "sudo chmod -R 777 #{target_dir}", ssh_info)
           remove_dir = build_ssh_command(ssh_opts, "sudo rm -rf #{target_files}", ssh_info) if delete
 
         elsif opts[:direction] == :download
@@ -64,21 +67,26 @@ module VagrantPlugins
           source = "#{ssh_info[:username]}@#{ssh_info[:host]}:#{source_files}"
           source = "#{source}/*" if has_trailing_slash_source
 
-          # Create target directory if needed
+          # Create local target directory without sudo
           target = target_files
-          parent_dir = File.dirname(target)
-          make_dir = "mkdir -p #{parent_dir}"
+          target_dir = target_files
+          target_dir = File.dirname(target_files) unless File.directory?(target_files) || has_trailing_slash_target
+          make_dir = "mkdir -p #{target_dir}"
         end
 
-        # Execute commands
-        execute_command(machine, remove_dir, delete, nil, opts) if delete
-        execute_command(machine, make_dir, false, nil, opts)
-        execute_command(machine, change_ownership, false, nil, opts) if opts[:direction] == :upload
-        execute_command(machine, change_permissions, false, nil, opts) if opts[:direction] == :upload
+        # Execute commands with proper error messages
+        execute_command(machine, remove_dir, true, 'scp_sync_error_delete_directory', opts) if delete
+        execute_command(machine, make_dir, true, 'scp_sync_error_make_directory', opts)
+
+        # For upload, ensure remote directory permissions
+        if opts[:direction] == :upload || opts[:direction].nil?
+          execute_command(machine, change_ownership, true, 'scp_sync_error_change_ownership_directory', opts)
+          execute_command(machine, change_permissions, true, 'scp_sync_error_change_permissions_directory', opts)
+        end
 
         # Build and execute the scp command
         synchronize = build_scp_command(scp_path, ssh_opts, source, target)
-        execute_command(machine, synchronize, true, 'scp_sync_folder', opts)
+        execute_command(machine, synchronize, true, 'scp_sync_folder_error', opts)
       end
 
       def self.expand_path(path, machine)
@@ -86,17 +94,6 @@ module VagrantPlugins
         Vagrant::Util::Platform.fs_real_path(expanded_path).to_s
       end
 
-      def self.build_ssh_options(ssh_info)
-        opts = %w[
-          -o StrictHostKeyChecking=no
-          -o UserKnownHostsFile=/dev/null
-          -o LogLevel=ERROR
-        ]
-        opts << "-o port=#{ssh_info[:port]}"
-        opts << ssh_info[:private_key_path].map { |k| "-i #{k}" }.join(' ')
-        opts
-      end
-
       def self.build_scp_options(opts)
         opts[:scp__args] ? Array(opts[:scp__args]).dup : ['--verbose']
       end
@@ -141,7 +138,7 @@ module VagrantPlugins
               stderr: stderr
       end
 
-      private_class_method :expand_path, :build_ssh_options, :build_scp_options,
+      private_class_method :expand_path, :build_scp_options,
                            :build_ssh_command, :build_scp_command, :execute_command,
                            :execute_command_with_output, :raise_scp_error
     end

data/lib/vagrant-scp-sync/ssh_options.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module VagrantPlugins
+  module ScpSync
+    # Helper class for building SSH options
+    class SshOptions
+      def self.build(ssh_info)
+        opts = %w[
+          -o StrictHostKeyChecking=no
+          -o UserKnownHostsFile=/dev/null
+          -o LogLevel=ERROR
+        ]
+        opts << "-o port=#{ssh_info[:port]}"
+        opts << ssh_info[:private_key_path].map { |k| "-i #{k}" }.join(' ')
+        opts
+      end
+    end
+  end
+end

data/lib/vagrant-scp-sync/version.rb

@@ -2,7 +2,7 @@
 
 module Vagrant
   module ScpSync
-    VERSION = '0.5.15'
+    VERSION = '0.5.16'
     NAME = 'vagrant-scp-sync'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-scp-sync
 version: !ruby/object:Gem::Version
-  version: 0.5.15
+  version: 0.5.16
 platform: ruby
 authors:
 - Mark Gilbert
@@ -143,6 +143,7 @@ files:
 - lib/vagrant-scp-sync/command/scp.rb
 - lib/vagrant-scp-sync/errors.rb
 - lib/vagrant-scp-sync/plugin.rb
+- lib/vagrant-scp-sync/ssh_options.rb
 - lib/vagrant-scp-sync/synced_folder.rb
 - lib/vagrant-scp-sync/version.rb
 - locales/en.yml