net-ping 1.3.7 → 1.4.0

data/CHANGES

@@ -1,3 +1,17 @@
+== 1.4.0 - 14-Feb-2011
+* Added the redirect_limit accessor for the Net::Ping::HTTP class. This caps
+  the number of redirects that are allowed before a ping is considered a
+  failure. The default is 5.
+* Changed the way in which redirects are detected. Previously I was checking
+  against the response class, but decided this was unreliable. I now check the
+  response code itself. Any code in the 300 range is considered a redirect.
+* The default port for the Net::Ping::HTTP class is no longer explicitly set
+  to 80, but instead uses the result of the URI.parse(uri).port method by
+  default. Note that you can still override the port if you explicitly provide
+  one in the constructor.
+* The Net::Ping::HTTP class now handles https URI's properly. Note that you may
+  get a warning about peer certificates.
+
 == 1.3.7 - 14-Oct-2010
 * Wrapped the ICMP ping in a while loop that breaks on success to prevent
   the ping from failing too early. Thanks go to Benny Holmgren for the

data/README

@@ -2,11 +2,9 @@
    A collection of classes that provide different ways to ping computers.
 
 == Prerequisites
-   * Ruby 1.8.0 or later
    * The win32-open3 (1.8.x) and windows-pr libraries are required on
      MS Windows when using the Net::Ping::External class unless you're
      using JRuby.
-   * Windows 2000 or later is required to use the Ping::WMI class.
 
 == Installation
    gem install net-ping
@@ -36,8 +34,8 @@
    
 == Known Issues
    Older versions of Ruby 1.9.x may not work with UDP pings.
-   As of JRuby 1.5.1, UDP pings will return false positives because of an
-   incorrect error class being raised. See JRuby-4896.
+   Older versions of JRuby will return false positives in UDP pings
+   because of an incorrect error class being raised. See JRuby-4896.
 
 == License
    Artistic 2.0

data/Rakefile

@@ -1,10 +1,13 @@
 require 'rake'
+require 'rake/clean'
 require 'rake/testtask'
 include Config
 
+CLEAN.include("**/*.gem", "**/*.rbc")
+
 namespace 'gem' do
   desc 'Create the net-ping gem'
-  task :create do
+  task :create => [:clean] do
     spec = eval(IO.read('net-ping.gemspec'))
     Gem::Builder.new(spec).build
   end

data/lib/net/ping.rb

@@ -10,7 +10,7 @@ require File.join(File.dirname(__FILE__), 'ping/icmp')
 require File.join(File.dirname(__FILE__), 'ping/external')
 require File.join(File.dirname(__FILE__), 'ping/http')
 
-if Config::CONFIG['host_os'] =~ /msdos|mswin|cygwin|mingw|win32/i &&
+if Config::CONFIG['host_os'] =~ /msdos|mswin|cygwin|mingw|win32|windows/i &&
   RUBY_PLATFORM != 'java'
 then
   require File.join(File.dirname(__FILE__), 'ping/wmi')

data/lib/net/ping/external.rb

@@ -1,7 +1,7 @@
 require 'rbconfig'
 require File.join(File.dirname(__FILE__), 'ping')
 
-if Config::CONFIG['host_os'] =~ /mswin|win32|msdos|cygwin|mingw/i &&
+if Config::CONFIG['host_os'] =~ /mswin|win32|msdos|cygwin|mingw|windows/i &&
   RUBY_PLATFORM != 'java'
 then
   if RUBY_VERSION.to_f < 1.9
@@ -18,7 +18,7 @@ module Net
   # The Ping::External class encapsulates methods for external (system) pings.
   class Ping::External < Ping
 
-    CWINDOWS = Config::CONFIG['host_os'] =~ /mswin|win32|msdos|cygwin|mingw/i &&
+    CWINDOWS = Config::CONFIG['host_os'] =~ /mswin|win32|msdos|cygwin|mingw|windows/i &&
       RUBY_PLATFORM != 'java'
       
     if CWINDOWS

data/lib/net/ping/http.rb

@@ -1,34 +1,41 @@
 require File.join(File.dirname(__FILE__), 'ping')
 require 'net/http'
+require 'net/https'
 require 'uri'
-require 'rbconfig'
 
 # Force non-blocking Socket.getaddrinfo on Unix systems. Do not use on
 # Windows because it (ironically) causes blocking problems.
-unless Config::CONFIG['host_os'] =~ /mswin|win32|msdos|cygwin|mingw/i
+unless File::ALT_SEPARATOR
   require 'resolv-replace'
 end
 
 # The Net module serves as a namespace only.
 module Net
-   
+
   # The Ping::HTTP class encapsulates methods for HTTP pings.
   class Ping::HTTP < Ping
-      
+
     # By default an http ping will follow a redirect and give you the result
     # of the final URI.  If this value is set to false, then it will not
     # follow a redirect and will return false immediately on a redirect.
-    # 
+    #
     attr_accessor :follow_redirect
 
-    # Sets the user agent used for the HTTP request to a custom one.
+    # The maximum number of redirects allowed. The default is 5.
+    attr_accessor :redirect_limit
+
+    # The user agent used for the HTTP request. The default is nil.
     attr_accessor :user_agent
 
-    # Creates and returns a new Ping::HTTP object.  Note that the default
-    # port for Ping::HTTP is 80.
-    # 
-    def initialize(uri=nil, port=80, timeout=5)
+    # Creates and returns a new Ping::HTTP object. The default port is the
+    # port associated with the URI. The default timeout is 5 seconds.
+    #
+    def initialize(uri=nil, port=nil, timeout=5)
       @follow_redirect = true
+      @redirect_limit  = 5
+
+      port ||= URI.parse(uri).port if uri
+
       super(uri, port, timeout)
     end
 
@@ -37,7 +44,7 @@ module Net
     # successful and true is returned.  Otherwise, false is returned
     # and the Ping::HTTP#exception method should contain a string
     # indicating what went wrong.
-    # 
+    #
     # If the HTTP#follow_redirect accessor is set to true (which it is
     # by default) and a redirect occurs during the ping, then the
     # HTTP#warning attribute is set to the redirect message, but the
@@ -45,7 +52,7 @@ module Net
     # response is considered a failed ping.
     #
     # If no file or path is specified in the URI, then '/' is assumed.
-    # 
+    #
     def ping(host = @host)
       super(host)
       bool = false
@@ -59,9 +66,10 @@ module Net
         headers = { }
         headers["User-Agent"] = user_agent unless user_agent.nil?
         Timeout.timeout(@timeout) do
-          Net::HTTP.new(uri.host, @port).start do |http|
-            response = http.request_get(uri_path, headers)
-          end
+          http = Net::HTTP.new(uri.host, uri.port)
+          http.use_ssl = (uri.scheme == 'https')
+          request = Net::HTTP::Get.new(uri_path)
+          response = http.start{ |h| h.request(request) }
         end
       rescue Exception => err
         @exception = err.message
@@ -71,18 +79,25 @@ module Net
         else
           if @follow_redirect
             @warning = response.message
-                
-            while response.is_a?(Net::HTTPRedirection)
-              redirect = URI.parse(response['location']) 
-              redirect = uri + redirect if redirect.relative?                     
+            rlimit = 0
+
+            # Any response code in the 300 range is a redirect
+            while response.code.to_i >= 300 && response.code.to_i < 400
+              if rlimit >= redirect_limit
+                @exception = "Redirect limit exceeded"
+                break
+              end
+              redirect = URI.parse(response['location'])
+              redirect = uri + redirect if redirect.relative?
               response = Net::HTTP.get_response(redirect.host, redirect.path, @port)
+              rlimit += 1
             end
-                  
+
             if response.is_a?(Net::HTTPSuccess)
               bool = true
             else
               @warning = nil
-              @exception = response.message
+              @exception ||= response.message
             end
           else
             @exception = response.message

data/lib/net/ping/ping.rb

@@ -4,13 +4,13 @@ require 'timeout'
 # The Net module serves as a namespace only.
 #
 module Net
-   
+
    # The Ping class serves as an abstract base class for all other Ping class
    # types. You should not instantiate this class directly.
    #
    class Ping
       # The version of the net-ping library.
-      VERSION = '1.3.7'
+      VERSION = '1.4.0'
 
       # The host to ping. In the case of Ping::HTTP, this is the URI.
       attr_accessor :host
@@ -44,7 +44,7 @@ module Net
       # The default constructor for the Net::Ping class.  Accepts an optional
       # +host+, +port+ and +timeout+.  The port defaults to your echo port, or
       # 7 if that happens to be undefined.  The default timeout is 5 seconds.
-      # 
+      #
       # The host, although optional in the constructor, must be specified at
       # some point before the Net::Ping#ping method is called, or else an
       # ArgumentError will be raised.
@@ -64,7 +64,7 @@ module Net
 
          yield self if block_given?
       end
-      
+
       # The default interface for the Net::Ping#ping method.  Each subclass
       # should call super() before continuing with their own implementation in
       # order to ensure that the @exception and @warning instance variables
@@ -81,7 +81,7 @@ module Net
          @exception = nil
          @warning   = nil
       end
-      
+
       alias ping? ping
       alias pingecho ping
    end

data/net-ping.gemspec

@@ -3,7 +3,7 @@ require 'rbconfig'
 
 Gem::Specification.new do |gem|
   gem.name      = 'net-ping'
-  gem.version   = '1.3.7'
+  gem.version   = '1.4.0'
   gem.license   = 'Artistic 2.0'
   gem.author    = 'Daniel J. Berger'
   gem.email     = 'djberg96@gmail.com'
@@ -16,10 +16,10 @@ Gem::Specification.new do |gem|
   gem.rubyforge_project = 'shards'
   gem.extra_rdoc_files  = ['README', 'CHANGES', 'doc/ping.txt']
 
-  gem.add_development_dependency('test-unit', '>= 2.0.9')
+  gem.add_development_dependency('test-unit', '>= 2.1.2')
 
   # These dependencies are for Net::Ping::External
-  if Config::CONFIG['host_os'] =~ /mswin|dos|win32|cygwin|mingw/i &&
+  if Config::CONFIG['host_os'] =~ /mswin|dos|win32|windows|cygwin|mingw/i &&
     RUBY_PLATFORM != 'java'
   then
     gem.platform = Gem::Platform::CURRENT

data/test/test_net_ping.rb

@@ -21,6 +21,6 @@ end
 
 class TC_Net_Ping < Test::Unit::TestCase
   def test_net_ping_version
-    assert_equal('1.3.7', Net::Ping::VERSION)
+    assert_equal('1.4.0', Net::Ping::VERSION)
   end
 end

data/test/test_net_ping_http.rb

@@ -13,7 +13,7 @@ require 'net/ping/http'
 class TC_Net_Ping_HTTP < Test::Unit::TestCase
   def setup
     @uri  = 'http://www.google.com/index.html'
-    @http = Net::Ping::HTTP.new(@uri, 80, 30)      
+    @http = Net::Ping::HTTP.new(@uri, 80, 30)
     @bad  = Net::Ping::HTTP.new('http://www.blabfoobarurghxxxx.com') # One hopes not
   end
 
@@ -108,11 +108,58 @@ class TC_Net_Ping_HTTP < Test::Unit::TestCase
     assert_nil(@http.warning)
   end
 
+  test 'user_agent accessor is defined' do
+    assert_respond_to(@http, :user_agent)
+    assert_respond_to(@http, :user_agent=)
+  end
+
+  test 'user_agent defaults to nil' do
+    assert_nil(@http.user_agent)
+  end
+
   test 'ping with user agent' do
     @http.user_agent = "KDDI-CA32"
     assert_true(@http.ping)
   end
 
+  test 'redirect_limit accessor is defined' do
+    assert_respond_to(@http, :redirect_limit)
+    assert_respond_to(@http, :redirect_limit=)
+  end
+
+  test 'redirect_limit defaults to 5' do
+    assert_equal(5, @http.redirect_limit)
+  end
+
+  test 'redirects succeed by default' do
+    @http = Net::Ping::HTTP.new("http://jigsaw.w3.org/HTTP/300/302.html")
+    assert_true(@http.ping)
+  end
+
+  test 'redirect fail if follow_redirect is set to false' do
+    @http = Net::Ping::HTTP.new("http://jigsaw.w3.org/HTTP/300/302.html")
+    @http.follow_redirect = false
+    assert_false(@http.ping)
+  end
+
+  test 'ping with redirect limit set to zero fails' do
+    @http = Net::Ping::HTTP.new("http://jigsaw.w3.org/HTTP/300/302.html")
+    @http.redirect_limit  = 0
+    assert_false(@http.ping)
+    assert_equal("Redirect limit exceeded", @http.exception)
+  end
+
+  test 'ping against https site defaults to port 443' do
+    @http = Net::Ping::HTTP.new("https://encrypted.google.com/")
+    assert_equal(443, @http.port)
+  end
+
+  # This will generate a warning. Nothing I can do about it.
+  test 'ping against https site works as expected' do
+    @http = Net::Ping::HTTP.new("https://encrypted.google.com/")
+    assert_true(@http.ping)
+  end
+
   def teardown
     @uri  = nil
     @http = nil

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: net-ping
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 7
   prerelease: false
   segments: 
   - 1
-  - 3
-  - 7
-  version: 1.3.7
+  - 4
+  - 0
+  version: 1.4.0
 platform: ruby
 authors: 
 - Daniel J. Berger
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-14 00:00:00 -06:00
+date: 2011-02-15 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -26,12 +26,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 29
+        hash: 15
         segments: 
         - 2
-        - 0
-        - 9
-        version: 2.0.9
+        - 1
+        - 2
+        version: 2.1.2
   type: :development
   version_requirements: *id001
 description: "    The net-ping library provides a ping interface for Ruby. It includes\n    separate TCP, HTTP, ICMP, UDP, WMI (for Windows) and external ping\n    classes.\n"