Merged in changes from 2.2.0

Author: SLsthompson

CHANGELOG.textile

@@ -6,6 +6,9 @@
 * Added a method to reboot servers.
 * The routine to retreive the open tickets on an account has been moved from the Ticket class. The set of open tickets is now a dynamic property of an account object.
 
+*2.2*
+* Added the ability to set a timout for network requests.  The timeout is given when a client is created by passing the :timeout hash parameter when creating a client. The value of the parameter is an integer number of seconds.
+
 *2.1.1*
 * Virtual server upgrades no longer raise exceptions
 * Formalized the RDoc documentation process. Added overview and welcome documentation and changed the README so it directs folks to the new documentation.

lib/softlayer/Client.rb

@@ -32,6 +32,11 @@ class Client
 
     # A string passsed as the value for the User-Agent header when requests are sent to SoftLayer API.
     attr_accessor :user_agent
+    
+    # An integer value (in seconds). The number of seconds to wait for HTTP requests to the network API
+    # until they timeout. This value can be nil in which case the timeout will be the default value for
+    # the library handling network communication (often 30 seconds)
+    attr_reader :network_timeout
 
     ##
     # The client class maintains an (optional) default client. The default client
@@ -60,6 +65,7 @@ def self.default_client=(new_default)
     # * <b>+:api_key+</b> - The API key used to authenticate the user with the API
     # * <b>+:enpoint_url+</b> - The API endpoint the client should connect to.  This defaults to API_PUBLIC_ENDPOINT
     # * <b>+:user_agent+</b> - A string that is passed along as the user agent when the client sends requests to the server
+    # * <b>+:timeout+</b> - An integer number of seconds to wait until network requests time out.  Corresponds to the network_timeout property of the client
     #
     # If these arguments are not provided then the client will try to locate them using other
     # sources including global variables, and the SoftLayer config file (if one exists)
@@ -79,6 +85,8 @@ def initialize(options = {})
       @endpoint_url = settings[:endpoint_url] || API_PUBLIC_ENDPOINT
 
       @user_agent = settings[:user_agent] || "softlayer_api gem/#{SoftLayer::VERSION} (Ruby #{RUBY_PLATFORM}/#{RUBY_VERSION})"
+      
+      @network_timeout = settings[:timeout] if settings.has_key?(:timeout)
 
       raise "A SoftLayer Client requires a username" if !@username || @username.empty?
       raise "A SoftLayer Client requires an api_key" if !@api_key || @api_key.empty?

lib/softlayer/Config.rb

@@ -38,6 +38,7 @@ module SoftLayer
   #      [softlayer]
   #      username = joeusername
   #      api_key = DEADBEEFBADF00D
+  #      timeout = 60
   #
   # = Environment Variables
   #
@@ -94,6 +95,7 @@ def Config.file_settings(*additional_files)
 						result[:username] = softlayer_section['username'] if softlayer_section['username']
 						result[:endpoint_url] = softlayer_section['endpoint_url'] if softlayer_section['endpoint_url']
 						result[:api_key] = softlayer_section['api_key'] if softlayer_section['api_key']
+						result[:timeout] = softlayer_section['timeout'] if softlayer_section['timeout']
 					end
 				end
 			end

lib/softlayer/ObjectMaskParser.rb

@@ -42,7 +42,7 @@ def parse(mask_string)
 
       recognize_token(@tokenizer, :eos, "Extraneous text after object mask: ")
 
-      if property && (property.name != "mask" && propertyName != "filterMask")
+      if property && (property.name != "mask" && property.name != "filterMask")
         raise ObjectMaskParserError, "Object Mask must begin with a 'mask' or 'filterMask' root property"
       end
 

lib/softlayer/Service.rb

@@ -296,9 +296,9 @@ def to_ary
 
     def xmlrpc_client()
       if !@xmlrpc_client
-        @xmlrpc_client = XMLRPC::Client.new2(URI.join(@client.endpoint_url,@service_name).to_s)
+        @xmlrpc_client = XMLRPC::Client.new2(URI.join(@client.endpoint_url,@service_name).to_s, nil, @client.network_timeout)
 
-        # this is a workaround for a bug in later versions of the XML-RPC client in Ruby Core.
+        # This is a workaround for a bug in later versions of the XML-RPC client in Ruby Core.
         # see https://bugs.ruby-lang.org/issues/8182
         @xmlrpc_client.http_header_extra = {
           "Accept-Encoding" => "identity",

spec/Client_spec.rb

@@ -95,6 +95,16 @@
     end.to raise_error
   end
 
+  it 'initializes by default with nil as the timeout' do
+    client = SoftLayer::Client.new(:username => 'fake_user', :api_key => 'fake_key', :endpoint_url => 'http://fakeurl.org/')
+    expect(client.network_timeout).to be_nil
+  end
+
+  it 'Accepts a timeout given as a config parameter' do
+    client = SoftLayer::Client.new(:username => 'fake_user', :api_key => 'fake_key', :endpoint_url => 'http://fakeurl.org/', :timeout => 60)
+    expect(client.network_timeout).to eq 60
+  end
+  
   it 'gets the default endpoint even if none is provided' do
     $SL_API_BASE_URL = nil
     client = SoftLayer::Client.new(:username => 'fake_user', :api_key => 'fake_key')

spec/ObjectMaskParser_spec.rb

@@ -38,6 +38,27 @@
     expect { result = subject.parse("mask[two,children], bob") }.to raise_error
   end
 
+  it "should parse a mask with fiterMask" do
+    result = nil
+    expect { result = subject.parse("filterMask.simple1") }.to_not raise_error
+
+    expect(result.name).to eq 'filterMask'
+    expect(result.children[0].name).to eq 'simple1'
+  end
+  
+  it "should parse a mask set with fiterMask" do
+    result = nil
+    expect { result = subject.parse("[filterMask.simple1, filterMask.simple2]") }.to_not raise_error
+
+    expect(result.count).to eq 2
+    expect(result[0].name).to eq 'filterMask'
+    expect(result[0].children.count).to eq 1
+    expect(result[0].children[0].name).to eq "simple1"
+
+    expect(result[1].name).to eq 'filterMask'
+    expect(result[1].children.count).to eq 1
+    expect(result[1].children[0].name).to eq "simple2"
+  end  
 end
 
 describe SoftLayer::ObjectMaskParser, "#parse_property_set" do

spec/Service_spec.rb

@@ -61,11 +61,23 @@
       client = SoftLayer::Client.new() # authentication is taken from the globals
       expect { SoftLayer::Service.new("SoftLayer_Account", :client => client, :username => "sample_username", :api_key => "blah") }.to raise_error(RuntimeError)
     end
-
   end #describe #new
 end
 
-describe SoftLayer::Service do
+describe SoftLayer::Service, "xmlrpc client" do
+  before(:each) do
+    SoftLayer::Service.send(:public, :xmlrpc_client)
+  end
+  
+  it "Constructs an XMLRPC client with a given timeout value based on the timeout of the client" do
+    client = SoftLayer::Client.new(:username => 'fake_user', :api_key => 'fake_key', :timeout => 60)
+    ticket_service = client[:Ticket]
+    xmlrpc = ticket_service.xmlrpc_client()
+    expect(xmlrpc.timeout).to eq 60
+  end
+end
+
+describe SoftLayer::Service, "parameter filters" do
   let (:service) do
     SoftLayer::Service.new("SoftLayer_Ticket", :username => "sample_username", :api_key => "blah")
   end