net-ftp 0.1.3 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a662c5a1b1201b6381d8c674ef0fcbe4a5a882ca023a24bd0606b607ec2a9cf
-  data.tar.gz: a89263fa37869d75365ffa13206244fae8efcaa2dcd8a24aa032ebdaf58bbbd4
+  metadata.gz: 6da1dd16a259cb0b9da3833be6087725451c66020a77484962ce512ebe76bd86
+  data.tar.gz: 5baf86565938970ff01dca9f6c85d8ae5afa57493ca96c35e8ccb30184f8fd96
 SHA512:
-  metadata.gz: cd09e0b10d7188943dcbdf8a7af4482d2798f4207fbdd5e42c44e6156ccb40b29e6f026b38c773d10b69e3549776ee79a266a1c7fb33315ad899faf0e25852f9
-  data.tar.gz: 76ca5bc18729d1ad8f0aaf732845a08ed6d21539d60db29bdadab43d9d3801f53772fd9312f1bd900fbd4e16f533b61b2c08cf60bfef87f4a3ebb7ced2e51630
+  metadata.gz: b634e82e54a09c1ad7174bafa5049f77f4878c5bf8b5d9fe4f43626c75d83831b91968a211075f5e36c7514c29d7dd2f8834134dc08f4f63def4b6e5a266b991
+  data.tar.gz: 4b124cb421ad550f889fd5dce46efb22cb726516a1e245131a4d0c5c125c0eb229b8041e952df0c85abed0a8e82697ea3116ef7c1eae1db7ebd09f45ea854355

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@v4
     - 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.3"
+    VERSION = "0.3.0"
     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
@@ -338,19 +356,14 @@ module Net
     # SOCKS_SERVER, then a SOCKSSocket is returned, else a Socket is
     # returned.
     def open_socket(host, port) # :nodoc:
-      if defined? SOCKSSocket and ENV["SOCKS_SERVER"]
-        @passive = true
-        Timeout.timeout(@open_timeout, OpenTimeout) do
+      return Timeout.timeout(@open_timeout, OpenTimeout) {
+        if defined? SOCKSSocket and ENV["SOCKS_SERVER"]
+          @passive = true
           SOCKSSocket.open(host, port)
+        else
+          Socket.tcp(host, port)
         end
-      else
-        begin
-          Socket.tcp host, port, nil, nil, connect_timeout: @open_timeout
-        rescue Errno::ETIMEDOUT #raise Net:OpenTimeout instead for compatibility with previous versions
-          raise Net::OpenTimeout, "Timeout to open TCP connection to "\
-          "#{host}:#{port} (exceeds #{@open_timeout} seconds)"
-        end
-      end
+      }
     end
     private :open_socket
 
@@ -378,19 +391,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 +410,8 @@ module Net
             @sock.close
             raise
           end
+        else
+          set_socket(BufferedSocket.new(@bare_sock, read_timeout: @read_timeout))
         end
       end
     end
@@ -430,9 +443,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 +456,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 +554,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 +1256,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 +1276,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 +1409,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 +1456,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,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: net-ftp
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.3.0
 platform: ruby
 authors:
 - Shugo Maeda
 autorequire:
-bindir: exe
+bindir: bin
 cert_chain: []
-date: 2021-07-07 00:00:00.000000000 Z
+date: 2023-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-protocol
@@ -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.0.dev
+rubygems_version: 3.5.0.dev
 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