omniauth / omniauth_openid_connect

MIT License
170 stars 187 forks source link

When calling access_token! make the first argument client_auth_method #123

Closed ssoulless closed 2 years ago

ssoulless commented 2 years ago

it could be problematic that on Rack::Oauth2::Client when it tries to set the client_auth_method https://github.com/nov/rack-oauth2/blob/9cb51cee480d00a7df45e8b940ac8ad20721d6a7/lib/rack/oauth2/client.rb#L137

It will always take the first argument as the default client_auth_method. Right now when calling access_token! on the client the first argument sent is scope instead of client_auth_method.