-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Oauth2 version 2.0.0 breaks omniauth-apple #85
Comments
@nhosoya how can we help get some progress on updating to support OAuth 2.0.0? Many of us using this are also using Facebook and Google strategies which work with the new version and it feels risky to not be able to update. |
Just tagging on a « me-too » to this @nhosoya |
nov
added a commit
to nov/omniauth-apple
that referenced
this issue
Sep 27, 2022
nov
added a commit
to nov/omniauth-apple
that referenced
this issue
Sep 27, 2022
Thank you ! |
Indeed, thank you!
El mar, 27 sept 2022 17:53, Scott Menor ***@***.***> escribió:
… Thank you !
—
Reply to this email directly, view it on GitHub
<#85 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADYXC4QXWJNEKWH2ZSHLBTWAK7YVANCNFSM56SZLW5Q>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Specifically this line in the Oauth2 2.0.0 release changelog breaks omniauth-apple gem:
"oauth-xx/oauth2#312 - BREAKING: Set :basic_auth as default for :auth_scheme instead of :request_body. This was default behavior before 1.3.0. (@tetsuya, @wy193777)
Apple get token call needs the "client_id" and "client_secret" in the body, which will only be sent if Oauth2 "auth_scheme" type is ":request_body"
Please provide a fix or advice how we can set the :auth_scheme to :request_body through omniquth options.
Thanks
The text was updated successfully, but these errors were encountered: