Fix Reauth Loop on Login Form; Load L10N Files#5
Fix Reauth Loop on Login Form; Load L10N Files#5mitcho merged 1 commit intomitcho:masterfrom jrchamp:master
Conversation
Remove reauth argument to avoid destruction of session immediately after login. Add load_plugin_textdomain() call so that language overrides are used.
|
Thanks for the PR. The l10n part looks perfectly fine. But can you explain the reauth issue? Under what circumstances would reauth be added? Does it actually clear the Shibboleth session or just the WP session? What is this a fix for? Thanks. |
|
I clicked logout and went back to the WP login page. The reauth parameter was already in the URL (from WordPress I assume) and was being pulled into the URL in the Shibboleth login link. Clicking "Login with Shibboleth" kept taking me back to the WP login page until I removed the reauth argument. |
|
Ok, I could see how that would happen. I can't see this causing much damage so I'm going to merge it. Logging out still works properly as well? |
|
Yeah, I didn't have any issues logging out (the WP page itself still has On Wednesday, June 11, 2014, Michael Yoshitaka Erlewine <
Jonathan Champ Email: [email protected] North Carolina State University |
Fix Reauth Loop on Login Form; Load L10N Files
Remove reauth argument to avoid destruction of session immediately after login. Add load_plugin_textdomain() call so that language overrides are used.