Skip to content

Commit

Permalink
Merge pull request #1 from dshorthouse/patch-1
Browse files Browse the repository at this point in the history
Remove query parameters from the callback_url
  • Loading branch information
jhoskins authored Jan 15, 2023
2 parents 3a43234 + 8a9289c commit 51c0173
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/omniauth/strategies/oauth2.rb
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ def pkce_token_params

def build_access_token
verifier = request.params["code"]
client.auth_code.get_token(verifier, {:redirect_uri => callback_url}.merge(token_params.to_hash(:symbolize_keys => true)), deep_symbolize(options.auth_token_params))
client.auth_code.get_token(verifier, {:redirect_uri => callback_url.split("?").first}.merge(token_params.to_hash(:symbolize_keys => true)), deep_symbolize(options.auth_token_params))
end

def deep_symbolize(options)
Expand Down

0 comments on commit 51c0173

Please sign in to comment.