psignoret / aad-sso-wordpress

Single Sign-on with Azure Active Directory (for WordPress)
Other
270 stars 78 forks source link

Too many redirects with WordPress 5.4 #234

Open Ssy3 opened 4 years ago

Ssy3 commented 4 years ago

I’m facing Too many redirects with WordPress 5.4 When I downgraded the WordPress to 5.3.2 Everything worked fine

keithnet commented 4 years ago

Same issue here. Hoping for a quick fix.

keithnet commented 4 years ago

I've done a bit more troubleshooting of this issue. the plugin works fine on some of my sites, but not others. I'm using WordPress 5.4 and just upgraded to 5.4.1 (and no difference).

The best way I've done troubleshooting is by disabling the automatic redirect. On the site that works, the link to sign in with SSO looks like this: https://login.windows.net/xxxxx/oauth2/authorize?response_type=code&scope=openid&domain_hint=&client_id=xxxx&resource=https%3A%2F%2Fgraph.microsoft.com&redirect_uri=https%3A%2F%2Fworking_site_domain.com%2Fwp-login.php&state=xxxx&nonce=xxxx

On the site that doesn't work, the link looks like this. If you click on it, it brings you right back to login page: https://site_domainname_here.com/wp-login.php?response_type=code&scope=openid&domain_hint=&client_id=xxxxx&resource=https%3A%2F%2Fgraph.microsoft.com&redirect_uri=https%3A%2F%2Fsite_domainname_here.com%2Fwp-login.php&state=xxxx&nonce=xxxx

I've reconfigured the broken site several times with the same exact settings as the function site. Even did brand new app registration and no luck.

virtazp commented 4 years ago

Hello, That's the same for me. I have to put as redirect https: //mysite/wp-login.php for this to work. If I put in redirection URI, my home page, it shows me error too many redirection. I did a test with the is_user_logged_in () function and indeed, it does not consider me logged in when the page loads.