decioferreira / omniauth-linkedin-oauth2

A LinkedIn OAuth2 strategy for OmniAuth.
MIT License
117 stars 147 forks source link

Oauth 2.0 breaking change fix #83

Open domschab23 opened 1 month ago

domschab23 commented 1 month ago

Uprading oauth2to version 2.0.9 https://rubygems.org/gems/oauth2/versions/2.0.9 introduces breaking changes with :auth_scheme. The default value in 2.X.X is :basic_auth but it used to be :request_body in 1.X.X https://github.com/oauth-xx/oauth2/blob/main/lib/oauth2/client.rb#L54 and notes here https://github.com/oauth-xx/oauth2/tree/main?tab=readme-ov-file#what-is-new-for-v20

Default value in 1.X.X https://gitlab.com/oauth-xx/oauth2/-/blob/1-4-stable/lib/oauth2/client.rb?ref_type=heads#L44

Some integrations still rely on request_body and linkedin is one of them so adding this

domschab23 commented 1 month ago

cc @decioferreira