SocialiteProviders / Providers

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

Azure: Add support for optional post_logout_redirect_uri #1173

Closed Tamas-hi closed 3 months ago

Tamas-hi commented 4 months ago

Support for optional post logout redirect uri

Issue

Right now, the library doesn't support the case when you don't want to redirect the user after a succesful logout. I needed this case, and I think it makes sense to make it available through this provider.

Solution

Set the $redirectUri as an optional parameter to keep it backward compatible. Post_logout_redirect_uri is meant to be optional also according to the OpenID (https://openid.net/specs/openid-connect-rpinitiated-1_0.html)

Tamas-hi commented 3 months ago

Thanks for merging my contribution. Can we expect a release in the near future so that we can use this functionality? Thanks!

atymic commented 3 months ago

Tagged sorry :)