net-ftp 0.1.4 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 21c0eba7e609e9e211574f9516bcdeecd08cfccd970ad52f098838cddbba2b23
-  data.tar.gz: f5e77d0c5fcf882ac5e9fa521fb29eed475fa5578886f1270c2b9047ba856b9d
+  metadata.gz: 18559c9c84d96f514739718efbda54e6fad17d4273091daa96439f8b2ca35088
+  data.tar.gz: 303c0fc5bfbf89f973b5dfc8adcab01353adbde0c1cc0a427ed5bcb545b680eb
 SHA512:
-  metadata.gz: a022858433f6ddb623e6cc5a1f7b3ff01c686dbcff371530064b71b0d6fb39eb7e31a107a23d9bacbf9f52ef8bf0aeb09d5a651cd338bc37ecc76591e0a06cf7
-  data.tar.gz: 416f37d820111d80b207ee426aa0007b237591d25662c6bb55de12483f0f8dd2d0f0ff0251d44dff594d5d46847c132b0ef97efa67ef12c4f756c712cb19650b
+  metadata.gz: d06b941364767d71f483fd1396328324b6c15c88072058f904ca69509f6e70c57c64a2c4c3e3ee041a23aaa5d83b598bcf16061e8b13b76056d405d64fb349de
+  data.tar.gz: 13234a36e3e7ab0d0026fe84c4ab42b61b3f7ca0354a78dbc98dcbee2e8402d4130658c327c6e1e2124096b4b1faa0092fdb8e159baef060b801746531120e71

data/.github/dependabot.yml

@@ -0,0 +1,6 @@
+version: 2
+updates:
+  - package-ecosystem: 'github-actions'
+    directory: '/'
+    schedule:
+      interval: 'weekly'

data/.github/workflows/test.yml

@@ -7,11 +7,11 @@ jobs:
     name: build (${{ matrix.ruby }} / ${{ matrix.os }})
     strategy:
       matrix:
-        ruby: [ '3.0', 2.7, 2.6, 2.5, 2.4, head ]
+        ruby: [ head, '3.1', '3.0', '2.7', '2.6' ]
         os: [ ubuntu-latest, macos-latest ]
     runs-on: ${{ matrix.os }}
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:

data/lib/net/ftp.rb

@@ -85,7 +85,7 @@ module Net
     end
 
     # :stopdoc:
-    VERSION = "0.1.4"
+    VERSION = "0.2.1"
     FTP_PORT = 21
     CRLF = "\r\n"
     DEFAULT_BLOCKSIZE = BufferedIO::BUFSIZE
@@ -106,6 +106,11 @@ module Net
     # to +$stdout+.  Default: +false+.
     attr_accessor :debug_mode
 
+    # Sets or retrieves the output stream for debugging.
+    # Output stream will be used only when +debug_mode+ is set to true.
+    # The default value is +$stdout+.
+    attr_accessor :debug_output
+
     # Sets or retrieves the +resume+ status, which decides whether incomplete
     # transfers are resumed or restarted.  Default: +false+.
     attr_accessor :resume
@@ -195,6 +200,8 @@ module Net
     #             as parameters.
     # private_data_connection::  If true, TLS is used for data connections.
     #                            Default: +true+ when +options+[:ssl] is true.
+    # implicit_ftps::  If true, TLS is established on initial connection.
+    #                  Default: +false+
     # username::  Username for login.  If +options+[:username] is the string
     #             "anonymous" and the +options+[:password] is +nil+,
     #             "anonymous@" is used as a password.
@@ -248,6 +255,11 @@ module Net
         else
           @private_data_connection = options[:private_data_connection]
         end
+        if options[:implicit_ftps].nil?
+          @implicit_ftps = false
+        else
+          @implicit_ftps = options[:implicit_ftps]
+        end
       else
         @ssl_context = nil
         if options[:private_data_connection]
@@ -255,6 +267,11 @@ module Net
             "private_data_connection can be set to true only when ssl is enabled"
         end
         @private_data_connection = false
+        if options[:implicit_ftps]
+          raise ArgumentError,
+                "implicit_ftps can be set to true only when ssl is enabled"
+        end
+        @implicit_ftps = false
       end
       @binary = true
       if options[:passive].nil?
@@ -267,6 +284,7 @@ module Net
       else
         @debug_mode = options[:debug_mode]
       end
+      @debug_output = $stdout
       @resume = false
       @bare_sock = @sock = NullSocket.new
       @logged_in = false
@@ -378,19 +396,17 @@ module Net
     # <tt>Errno::ECONNREFUSED</tt>) if the connection cannot be established.
     #
     def connect(host, port = FTP_PORT)
-      if @debug_mode
-        print "connect: ", host, ", ", port, "\n"
-      end
+      debug_print "connect: #{host}:#{port}"
       synchronize do
         @host = host
         @bare_sock = open_socket(host, port)
-        @sock = BufferedSocket.new(@bare_sock, read_timeout: @read_timeout)
-        voidresp
         if @ssl_context
           begin
-            voidcmd("AUTH TLS")
-            ssl_sock = start_tls_session(@bare_sock)
-            @sock = BufferedSSLSocket.new(ssl_sock, read_timeout: @read_timeout)
+            unless @implicit_ftps
+              set_socket(BufferedSocket.new(@bare_sock, read_timeout: @read_timeout))
+              voidcmd("AUTH TLS")
+            end
+            set_socket(BufferedSSLSocket.new(start_tls_session(@bare_sock), read_timeout: @read_timeout), @implicit_ftps)
             if @private_data_connection
               voidcmd("PBSZ 0")
               voidcmd("PROT P")
@@ -399,6 +415,8 @@ module Net
             @sock.close
             raise
           end
+        else
+          set_socket(BufferedSocket.new(@bare_sock, read_timeout: @read_timeout))
         end
       end
     end
@@ -430,9 +448,7 @@ module Net
     # Ensures that +line+ has a control return / line feed (CRLF) and writes
     # it to the socket.
     def putline(line) # :nodoc:
-      if @debug_mode
-        print "put: ", sanitize(line), "\n"
-      end
+      debug_print "put: #{sanitize(line)}"
       if /[\r\n]/ =~ line
         raise ArgumentError, "A line must not contain CR or LF"
       end
@@ -445,9 +461,7 @@ module Net
     def getline # :nodoc:
       line = @sock.readline # if get EOF, raise EOFError
       line.sub!(/(\r\n|\n|\r)\z/n, "")
-      if @debug_mode
-        print "get: ", sanitize(line), "\n"
-      end
+      debug_print "get: #{sanitize(line)}"
       return line
     end
     private :getline
@@ -545,7 +559,6 @@ module Net
         host, port = parse227(sendcmd("PASV"))
       else
         host, port = parse229(sendcmd("EPSV"))
-        #     host, port = parse228(sendcmd("LPSV"))
       end
       return host, port
     end
@@ -1248,7 +1261,7 @@ module Net
     #
     def abort
       line = "ABOR" + CRLF
-      print "put: ABOR\n" if @debug_mode
+      debug_print "put: ABOR"
       @sock.send(line, Socket::MSG_OOB)
       resp = getmultiline
       unless ["426", "226", "225"].include?(resp[0, 3])
@@ -1268,7 +1281,7 @@ module Net
       if /[\r\n]/ =~ line
         raise ArgumentError, "A line must not contain CR or LF"
       end
-      print "put: #{line}\n" if @debug_mode
+      debug_print "put: #{line}"
       @sock.send(line + CRLF, Socket::MSG_OOB)
       return getresp
     end
@@ -1401,24 +1414,6 @@ module Net
     end
     private :parse227
 
-    # handler for response code 228
-    # (Entering Long Passive Mode)
-    #
-    # Returns host and port.
-    def parse228(resp) # :nodoc:
-      if !resp.start_with?("228")
-        raise FTPReplyError, resp
-      end
-      if m = /\(4,4,(?<host>\d+(?:,\d+){3}),2,(?<port>\d+,\d+)\)/.match(resp)
-        return parse_pasv_ipv4_host(m["host"]), parse_pasv_port(m["port"])
-      elsif m = /\(6,16,(?<host>\d+(?:,\d+){15}),2,(?<port>\d+,\d+)\)/.match(resp)
-        return parse_pasv_ipv6_host(m["host"]), parse_pasv_port(m["port"])
-      else
-        raise FTPProtoError, resp
-      end
-    end
-    private :parse228
-
     def parse_pasv_ipv4_host(s)
       return s.tr(",", ".")
     end
@@ -1466,6 +1461,13 @@ module Net
     end
     private :parse257
 
+    #
+    # Writes debug message to the debug output stream
+    #
+    def debug_print(msg)
+      @debug_output << msg + "\n" if @debug_mode && @debug_output
+    end
+
     # :stopdoc:
     class NullSocket
       def read_timeout=(sec)

data/net-ftp.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
   spec.summary       = %q{Support for the File Transfer Protocol.}
   spec.description   = %q{Support for the File Transfer Protocol.}
   spec.homepage      = "https://github.com/ruby/net-ftp"
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.6.0")
   spec.licenses      = ["Ruby", "BSD-2-Clause"]
 
   spec.metadata["homepage_uri"] = spec.homepage
@@ -25,10 +25,8 @@ Gem::Specification.new do |spec|
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
   spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
-    `git ls-files -z 2>/dev/null`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+    `git ls-files -z 2>/dev/null`.split("\x0").reject { |f| f.match(%r{^(bin|test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
   spec.add_dependency "net-protocol"

metadata

@@ -1,12 +1,12 @@
 --- !ruby/object:Gem::Specification
 name: net-ftp
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.2.1
 platform: ruby
 authors:
 - Shugo Maeda
 autorequire:
-bindir: exe
+bindir: bin
 cert_chain: []
 date: 2024-01-05 00:00:00.000000000 Z
 dependencies:
@@ -45,14 +45,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
 - ".github/workflows/test.yml"
 - ".gitignore"
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
-- bin/console
-- bin/setup
 - lib/net/ftp.rb
 - net-ftp.gemspec
 homepage: https://github.com/ruby/net-ftp
@@ -70,14 +69,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.27
+rubygems_version: 3.4.19
 signing_key:
 specification_version: 4
 summary: Support for the File Transfer Protocol.

data/bin/console

@@ -1,14 +0,0 @@
-#!/usr/bin/env ruby
-
-require "bundler/setup"
-require "net/ftp"
-
-# You can add fixtures and/or initialization code here to make experimenting
-# with your gem easier. You can also use a different console, if you like.
-
-# (If you use this, don't forget to add pry to your Gemfile!)
-# require "pry"
-# Pry.start
-
-require "irb"
-IRB.start(__FILE__)

data/bin/setup

@@ -1,8 +0,0 @@
-#!/usr/bin/env bash
-set -euo pipefail
-IFS=$'\n\t'
-set -vx
-
-bundle install
-
-# Do any other automated setup that you need to do here