Browse Source

Merge pull request #470 from yjaspar/master

Rails 5.2.1 session issues fixes
Rafael França 6 years ago
parent
commit
cf58ae0506
2 changed files with 1 additions and 1 deletions
  1. 1 0
      lib/shopify_api/session.rb
  2. 0 1
      test/session_test.rb

+ 1 - 0
lib/shopify_api/session.rb

@@ -51,6 +51,7 @@ module ShopifyAPI
       end
 
       def validate_signature(params)
+        params = (params.respond_to?(:to_unsafe_hash) ? params.to_unsafe_hash : params).with_indifferent_access
         return false unless signature = params[:hmac]
 
         calculated_signature = OpenSSL::HMAC.hexdigest(OpenSSL::Digest::SHA256.new(), secret, encoded_params_for_signature(params))

+ 0 - 1
test/session_test.rb

@@ -232,7 +232,6 @@ class SessionTest < Test::Unit::TestCase
     params = { 'a%3D1%26b' => '2%26c%3D3' }
     to_sign = 'a%253D1%2526b=2%2526c%253D3'
     params[:hmac] = generate_signature(to_sign)
-
     assert_equal true, ShopifyAPI::Session.validate_signature(params)
   end