bfs-scp 0.4.3 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b951d84b60d557d36e6876b958f08fd0f143a0a47e27d73c2b6ee8f94a72cbb9
-  data.tar.gz: 2e0936571a5c70fd0cbe00915b782542a98684348502b67e5df66f9093624568
+  metadata.gz: 806a4a94596c4967bf51c5017c05cc60444a65ef67c8f76e9eb13448d8989428
+  data.tar.gz: 465ff7c8af4987f2ac97311cfb8039d584193a60e5837dfca2ba7759986d87cb
 SHA512:
-  metadata.gz: 0db63bed39574e4b836d8a6b31207426dc66299a135d833d05eb55ce2430d84ad84aa40e1215fbb22e64d6c716689dafb6d0638b36ec4b526608dd968ee38bde
-  data.tar.gz: 263916e2d42690db955f967b8d38887aed77154aeaf9afa38f4ebde7956a1dc8cd76d360eb42998a4fd62ec2c7ebe9be4f6481b06af863096128d1c094efc88c
+  metadata.gz: 459ec003bfbd273b9aee278f8fd81533b5d0493d573eefa8c6f9303b9e8d1bb444216ab9bd9d1785b3c92ac0e3e60fce8233535374b9be59658b82c85c7ebe4a
+  data.tar.gz: 5b54beb2bd9b9bcfdd40e621adabbb71d34421eee4cddfbae59c0b6c2a32f5fe1156a0657db539880766c33d437d28d2adb458213f7d4eb50e25ffc240ac617a

data/bfs-scp.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- spec/*`.split("\n")
   s.require_paths = ['lib']
-  s.required_ruby_version = '>= 2.4.0'
+  s.required_ruby_version = '>= 2.5.0'
 
   s.add_dependency 'bfs', s.version
   s.add_dependency 'net-scp'

data/lib/bfs/bucket/scp.rb

@@ -10,7 +10,7 @@ module BFS
       class CommandError < RuntimeError
         attr_reader :status
 
-        def initialize(cmd, status, extra=nil)
+        def initialize(cmd, status, extra = nil)
           @status = status
           super ["Command '#{cmd}' exited with status #{status}", extra].join(': ')
         end
@@ -28,30 +28,30 @@ module BFS
       # @option opts [Integer] :keepalive_interval interval if keepalive enabled. Default: 300.
       # @option opts [Array<String>] :keys an array of file names of private keys to use for publickey and hostbased authentication.
       # @option opts [Boolean|Symbol] :verify_host_key specifying how strict host-key verification should be, either false, true, :very, or :secure.
-      def initialize(host, opts={})
+      def initialize(host, **opts)
         opts = opts.dup
         opts.keys.each do |key|
           val = opts.delete(key)
           opts[key.to_sym] = val unless val.nil?
         end
-        super(opts)
+        super(**opts)
 
         @prefix = opts.delete(:prefix)
-        @client = Net::SCP.start(host, nil, opts)
+        @client = Net::SCP.start(host, nil, **opts)
 
         if @prefix # rubocop:disable Style/GuardClause
           @prefix = norm_path(@prefix) + '/'
-          mkdir_p(@prefix)
+          mkdir_p abs_path(@prefix)
         end
       end
 
       # Lists the contents of a bucket using a glob pattern
-      def ls(pattern='**/*', _opts={})
-        prefix = @prefix || '.'
+      def ls(pattern = '**/*', **_opts)
+        prefix = @prefix ? abs_path(@prefix) : '/'
         Enumerator.new do |y|
           sh! 'find', prefix, '-type', 'f' do |out|
             out.each_line do |line|
-              path = trim_prefix(line.strip)
+              path = trim_prefix(norm_path(line.strip))
               y << path if File.fnmatch?(pattern, path, File::FNM_PATHNAME)
             end
           end
@@ -59,7 +59,7 @@ module BFS
       end
 
       # Info returns the object info
-      def info(path, _opts={})
+      def info(path, **_opts)
         full = full_path(path)
         path = norm_path(path)
         out  = sh! 'stat', '-c', '%s;%Z', full
@@ -71,12 +71,15 @@ module BFS
       end
 
       # Creates a new file and opens it for writing
-      def create(path, opts={}, &block)
+      # @option opts [String|Encoding] :encoding Custom file encoding.
+      # @option opts [Integer] :perm Custom file permission, default: 0600.
+      def create(path, **opts, &block)
         full = full_path(path)
+        perm = opts[:perm]
         enc  = opts.delete(:encoding) || @encoding
-        temp = BFS::TempWriter.new(path, encoding: enc) do |temp_path|
+        temp = BFS::TempWriter.new(path, perm: perm, encoding: enc) do |temp_path|
           mkdir_p File.dirname(full)
-          @client.upload!(temp_path, full)
+          @client.upload!(temp_path, full, preserve: true)
         end
         return temp unless block
 
@@ -88,7 +91,7 @@ module BFS
       end
 
       # Opens an existing file for reading
-      def open(path, opts={}, &block)
+      def open(path, **opts, &block)
         full = full_path(path)
         enc  = opts.delete(:encoding) || @encoding
         temp = Tempfile.new(File.basename(path), encoding: enc)
@@ -101,7 +104,7 @@ module BFS
       end
 
       # Deletes a file.
-      def rm(path, _opts={})
+      def rm(path, **_opts)
         path = full_path(path)
         sh! 'rm', '-f', path
       end
@@ -110,7 +113,7 @@ module BFS
       #
       # @param [String] src The source path.
       # @param [String] dst The destination path.
-      def cp(src, dst, _opts={})
+      def cp(src, dst, **_opts)
         full_src = full_path(src)
         full_dst = full_path(dst)
 
@@ -124,7 +127,7 @@ module BFS
       #
       # @param [String] src The source path.
       # @param [String] dst The destination path.
-      def mv(src, dst, _opts={})
+      def mv(src, dst, **_opts)
         full_src = full_path(src)
         full_dst = full_path(dst)
 
@@ -141,6 +144,15 @@ module BFS
 
       private
 
+      def abs_path(path)
+        path = "/#{path}" unless path.start_with?('~/', './')
+        path
+      end
+
+      def full_path(*)
+        abs_path(super)
+      end
+
       def mkdir_p(path)
         sh! 'mkdir', '-p', path
       end
@@ -186,5 +198,5 @@ BFS.register('scp', 'ssh') do |url|
   opts[:password] ||= CGI.unescape(url.password) if url.password
   opts[:port] ||= url.port if url.port
 
-  BFS::Bucket::SCP.new url.host, opts
+  BFS::Bucket::SCP.new(url.host, **opts)
 end

data/spec/bfs/bucket/scp_spec.rb

@@ -13,15 +13,36 @@ run_spec = \
   end
 
 RSpec.describe BFS::Bucket::SCP, if: run_spec do
-  subject { described_class.new sandbox[:host], sandbox[:opts].merge(prefix: SecureRandom.uuid) }
-  after   { subject.close }
+  context 'absolute' do
+    subject { described_class.new sandbox[:host], **sandbox[:opts].merge(prefix: SecureRandom.uuid) }
+    after   { subject.close }
 
-  it_behaves_like 'a bucket',
-    content_type: false,
-    metadata: false
+    it_behaves_like 'a bucket', content_type: false, metadata: false
+  end
+
+  context 'relative' do
+    subject { described_class.new sandbox[:host], **sandbox[:opts].merge(prefix: "~/#{SecureRandom.uuid}") }
+    after   { subject.close }
+
+    it_behaves_like 'a bucket', content_type: false, metadata: false
+  end
 
   it 'should resolve from URL' do
     bucket = BFS.resolve('scp://root:root@127.0.0.1:7022')
     expect(bucket).to be_instance_of(described_class)
   end
+
+  it 'should handle absolute and relative paths' do
+    abs = BFS::Blob.new("scp://root:root@127.0.0.1:7022/#{SecureRandom.uuid}/file.txt")
+    abs.create {|w| w.write 'absolute' }
+
+    rel = BFS::Blob.new("scp://root:root@127.0.0.1:7022/~/#{SecureRandom.uuid}/file.txt")
+    rel.create {|w| w.write 'relative' }
+
+    expect(abs.read).to eq('absolute')
+    expect(rel.read).to eq('relative')
+
+    abs.close
+    rel.close
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bfs-scp
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.6.1
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-09-19 00:00:00.000000000 Z
+date: 2020-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bfs
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.3
+        version: 0.6.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.3
+        version: 0.6.1
 - !ruby/object:Gem::Dependency
   name: net-scp
   requirement: !ruby/object:Gem::Requirement
@@ -60,14 +60,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.4.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: SCP/SSH adapter for bfs