oauth 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5b45dc41aae4a7483d4dc3f37231ed79084a7c8e
-  data.tar.gz: c82a432eb7f1a7ab2cb704eedd1ae9da15daed2c
+  metadata.gz: 4e52bba31bed6b4a6bb18b934ce067a3e5a5f9c0
+  data.tar.gz: 0c64d7eec3f2ff1e1d257b9762b93fe30512032b
 SHA512:
-  metadata.gz: 6b3a8ebb69c99d98762a66ddaec9074ee6161824405f04dab2d057e8059fa6cb298985c951479980c41800508a11edbc439a38602e39384add3c2a9424be8975
-  data.tar.gz: 65f7255a19a1def380575faa1a97d95ec78821fcf687ff8f914a10d367dcc6e05d1681e4a7b9106973e5a50618457e35d8c1b07adbf21919e668345ca18143ab
+  metadata.gz: ab1329b4b6a666b9eb80b46337727beeb1b2825f3358fc9647e5dda57c3a1c3fe2f6e9a427f8672dca7ae36d11b6aabb6278891f56b6a13ff4fa2f26aee3d023
+  data.tar.gz: 2c871f1404d9c02457ac3a08becf5a6e8e56c9e68f3a15b3bfbd9841a23a9f7d6ce2d808a921f68ffb51454861e990305998c430c3c197b2233e5fd17cc4d65d

data/lib/oauth/request_proxy/action_controller_request.rb

@@ -67,7 +67,7 @@ module OAuth::RequestProxy
 
       params.
         join('&').split('&').
-        reject(&:blank?).
+        reject { |s| s.match(/\A\s*\z/) }.
         map { |p| p.split('=').map{|esc| CGI.unescape(esc)} }.
         reject { |kv| kv[0] == 'oauth_signature'}
     end

data/lib/oauth/request_proxy/base.rb

@@ -146,7 +146,7 @@ module OAuth::RequestProxy
       if uri = request.env['REQUEST_URI']
         uri.split('?', 2)[1].nil?
       else
-        request.query_string.blank?
+        request.query_string.match(/\A\s*\z/)
       end
     end
 

data/lib/oauth/tokens/request_token.rb

@@ -26,11 +26,11 @@ module OAuth
     # construct an authorization url
     def build_authorize_url(base_url, params)
       uri = URI.parse(base_url.to_s)
-      if(!uri.query.blank? && !params.empty?)
-	uri.query += "&"
-      end
+      queries = {}
+      queries = Hash[URI.decode_www_form(uri.query)] if uri.query
       # TODO doesn't handle array values correctly
-      uri.query = params.map { |k,v| [k, CGI.escape(v)] * "=" } * "&"
+      queries.merge!(params) if params
+      uri.query = URI.encode_www_form(queries) if !queries.empty?
       uri.to_s
     end
   end

data/lib/oauth/version.rb

@@ -1,3 +1,3 @@
 module OAuth
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 end

data/test/cases/spec/1_0-final/test_normalize_request_parameters.rb

@@ -27,13 +27,13 @@ require File.expand_path('../../../oauth_case', __FILE__)
 class NormalizeRequestParametersTest < OAuthCase
 
   def test_parameters_for_signature
-    params={'a'=>1, 'c'=>'hi there', 'f'=>'25', 'f'=>'50', 'f'=>'a', 'z'=>'p', 'z'=>'t'}
+    params={"a"=>1, "c"=>"hi there", "f"=>"a", "z"=>"t"}
     assert_equal params,request(params).parameters_for_signature
   end
 
 
   def test_parameters_for_signature_removes_oauth_signature
-    params={'a'=>1, 'c'=>'hi there', 'f'=>'25', 'f'=>'50', 'f'=>'a', 'z'=>'p', 'z'=>'t'}
+    params={"a"=>1, "c"=>"hi there", "f"=>"a", "z"=>"t"}
     assert_equal params,request(params.merge({'oauth_signature'=>'blalbla'})).parameters_for_signature
   end
 

data/test/test_net_http_client.rb

@@ -291,6 +291,10 @@ class NetHTTPClientTest < Minitest::Test
 
   protected
 
+  def assert_no_match exp, act, msg = "Expected #{act.inspect} to not match #{exp.inspect}"
+    assert act !~ exp, msg
+  end
+
     def request_parameters_to_s
       @request_parameters.map { |k,v| "#{k}=#{v}" }.join("&")
     end

data/test/test_rest_client_request_proxy.rb

@@ -3,9 +3,9 @@ require File.expand_path('../test_helper', __FILE__)
 begin
   require 'oauth/request_proxy/rest_client_request'
   require 'rest-client'
-  
-  class RestlClientRequestProxyTest < Test::Unit::TestCase
-  
+
+  class RestlClientRequestProxyTest < Minitest::Test
+
     def test_that_proxy_simple_get_request_works
       request = ::RestClient::Request.new(method: :get, url: "http://example.com/test?key=value")
       request_proxy = OAuth::RequestProxy.proxy(request, {:uri => 'http://example.com/test?key=value'})
@@ -15,7 +15,7 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'GET', request_proxy.method
     end
-    
+
     def test_that_proxy_simple_post_request_works_with_arguments
       request = ::RestClient::Request.new(method: :post, url: "http://example.com/test")
       params = {'key' => 'value'}
@@ -26,7 +26,7 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'POST', request_proxy.method
     end
-    
+
     def test_that_proxy_simple_post_request_works_with_form_data
       request = ::RestClient::Request.new(method: :post, url: "http://example.com/test",
         payload: {'key' => 'value'},
@@ -38,7 +38,7 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'POST', request_proxy.method
     end
-    
+
     def test_that_proxy_simple_put_request_works_with_arguments
       request = ::RestClient::Request.new(method: :put, url: "http://example.com/test")
       params = {'key' => 'value'}
@@ -49,7 +49,7 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'PUT', request_proxy.method
     end
-    
+
     def test_that_proxy_simple_put_request_works_with_form_data
       request = ::RestClient::Request.new(method: :put, url: "http://example.com/test",
         payload: {'key' => 'value'},
@@ -61,7 +61,7 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'PUT', request_proxy.method
     end
-    
+
     def test_that_proxy_post_request_works_with_mixed_parameter_sources
       request = ::RestClient::Request.new(url: 'http://example.com/test?key=value',
         method: :post,
@@ -74,8 +74,8 @@ begin
       assert_equal 'http://example.com/test', request_proxy.normalized_uri
       assert_equal 'POST', request_proxy.method
     end
-    
+
   end
 rescue LoadError => e
     warn "! problem loading rest-client, skipping these tests: #{e}"
-end
+end

data/test/test_signature_hmac_sha1.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper', __FILE__)
 require 'oauth/signature/hmac/sha1'
 
-class SignatureHMACSHA1Test < Test::Unit::TestCase
+class SignatureHMACSHA1Test < Minitest::Test
   def test_that_verify_returns_true_when_the_request_signature_is_right
     request = OAuth::RequestProxy::MockRequest.new(
       'method' => 'POST',

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: oauth
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Pelle Braendgaard
@@ -15,7 +15,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-20 00:00:00.000000000 Z
+date: 2016-02-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -59,20 +59,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: jeweler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
@@ -91,30 +77,30 @@ dependencies:
   name: iconv
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.3
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.3
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
@@ -175,16 +161,16 @@ dependencies:
   name: curb
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.16
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.16
+        version: '0'
 description: 
 email: oauth-ruby@googlegroupspec.com
 executables:
@@ -268,7 +254,8 @@ files:
 - test/test_token.rb
 - test/test_typhoeus_request_proxy.rb
 homepage: 
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []