SocialiteProviders / Providers

A Collection of Providers for Laravel Socialite
https://socialiteproviders.com
MIT License
499 stars 438 forks source link

fix(okta): remove duplicate credentials #1150

Closed Tugzrida closed 7 months ago

Tugzrida commented 7 months ago

https://github.com/laravel/socialite/pull/684 added the client_id and client_secret to the Authorization header in addition to the request body, however Okta throws an error if multiple credentials are provided, so they must be removed from the body before sending the request.

driesvints commented 7 months ago

I've reverted the original PR.

lucasmichot commented 7 months ago

I've reverted the original PR.

Thanks for the head up @driesvints - I'll have a look at that

Tugzrida commented 7 months ago

Closing as https://github.com/laravel/socialite/pull/694 reverts the introduction of the bug