omniauth / omniauth-oauth2

An abstract OAuth2 strategy for OmniAuth.
MIT License
502 stars 304 forks source link

Fix Option `pkce_verifier` Cannot be Set Due to Overwrites #184

Open berniechiu opened 2 weeks ago

berniechiu commented 2 weeks ago

Hi @BobbyMcWho, thanks for this nice gem.

Description

The option :pkce_verifier, nil is available to be set in the OAuth Strategy configuration. However, it is being overwritten later phase in the #pkce_authorize_params function. We should only assign the default value when it is not set. This PR resolves the issue.