Skip to content

Fix Reauth Loop on Login Form; Load L10N Files#5

Merged
mitcho merged 1 commit intomitcho:masterfrom
jrchamp:master
Jun 12, 2014
Merged

Fix Reauth Loop on Login Form; Load L10N Files#5
mitcho merged 1 commit intomitcho:masterfrom
jrchamp:master

Conversation

@jrchamp
Copy link
Contributor

@jrchamp jrchamp commented Jun 9, 2014

Remove reauth argument to avoid destruction of session immediately after login. Add load_plugin_textdomain() call so that language overrides are used.

Remove reauth argument to avoid destruction of session immediately after login. Add load_plugin_textdomain() call so that language overrides are used.
@mitcho
Copy link
Owner

mitcho commented Jun 12, 2014

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.

@jrchamp
Copy link
Contributor Author

jrchamp commented Jun 12, 2014

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.

@mitcho
Copy link
Owner

mitcho commented Jun 12, 2014

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?

@jrchamp
Copy link
Contributor Author

jrchamp commented Jun 12, 2014

Yeah, I didn't have any issues logging out (the WP page itself still has
the reauth parameter set). This only modifies the Login with Shibboleth
link.

On Wednesday, June 11, 2014, Michael Yoshitaka Erlewine <
[email protected]> wrote:

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?


Reply to this email directly or view it on GitHub
#5 (comment).

Jonathan Champ
PHP Developer / Information Architect
Distance Education & Learning Technology Applications (DELTA)

Email: [email protected]
Phone: 919.513.0694

North Carolina State University
Campus Box 7113, Venture II, Suite 500
Raleigh, NC 27695

mitcho added a commit that referenced this pull request Jun 12, 2014
Fix Reauth Loop on Login Form; Load L10N Files
@mitcho mitcho merged commit 5321044 into mitcho:master Jun 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants