Merge pull request #119 from oauth-xx/reimplement-82-debug-output-option

Reimplementing #82 - Debug Output Option

Author: ch1ago

lib/oauth/consumer.rb

@@ -43,6 +43,13 @@ class Consumer
       # Add a custom ca_file for consumer
       # :ca_file       => '/etc/certs.pem'
 
+      # Possible values:
+      #
+      # nil, false - no debug output
+      # true - uses $stdout
+      # some_value - uses some_value
+      :debug_output => nil,
+
       :oauth_version => "1.0"
     }
 
@@ -87,6 +94,18 @@ def http_method
       @http_method ||= @options[:http_method] || :post
     end
 
+    def debug_output
+      @debug_output ||= begin
+        case @options[:debug_output]
+        when nil, false
+        when true
+          $stdout
+        else
+          @options[:debug_output]
+        end
+      end
+    end
+
     # The HTTP object for the site. The HTTP Object is what you get when you do Net::HTTP.new
     def http
       @http ||= create_http
@@ -322,6 +341,8 @@ def create_http(_url = nil)
       http_object.read_timeout = http_object.open_timeout = @options[:timeout] || 30
       http_object.open_timeout = @options[:open_timeout] if @options[:open_timeout]
       http_object.ssl_version = @options[:ssl_version] if @options[:ssl_version]
+      http_object.set_debug_output(debug_output) if debug_output
+
       http_object
     end
 

test/test_consumer.rb

@@ -38,6 +38,7 @@ def test_initializer
     assert_equal "http://blabla.bla/oauth/example/authorize.php",@consumer.authorize_url
     assert_equal :header,@consumer.scheme
     assert_equal :get,@consumer.http_method
+    assert_nil   @consumer.debug_output
   end
 
    def test_defaults
@@ -58,6 +59,28 @@ def test_defaults
     assert_equal "http://twitter.com/oauth/authorize",@consumer.authorize_url
     assert_equal :header,@consumer.scheme
     assert_equal :post,@consumer.http_method
+    assert_nil   @consumer.debug_output
+  end
+
+  def test_debug_output_true
+    @consumer=OAuth::Consumer.new(
+      "key",
+      "secret",
+      {
+          :debug_output=>true
+      })
+    assert_equal $stdout,@consumer.debug_output
+  end
+
+  def test_debug_output
+    stringio = StringIO.new
+    @consumer=OAuth::Consumer.new(
+      "key",
+      "secret",
+      {
+          :debug_output=>stringio
+      })
+    assert_equal stringio,@consumer.debug_output
   end
 
   def test_site_without_path