= Description = Auth with openid and other mechanisms that use redirects is broken. == Steps to reproduce == try to log in with openid. == Example == n/a == Component selection == * generic auth code == Details == || '''!MoinMoin Version''' || 1.9 as of today || || '''OS and Version''' || || || '''Python Version''' || || || '''Server Setup''' || || || '''Server Details''' || || || '''Language you are using the wiki in''' (set in the browser/UserPreferences) || || == Workaround == {{{ diff --git a/MoinMoin/auth/__init__.py b/MoinMoin/auth/__init__.py index bdcd7bc..fe470e7 100644 --- a/MoinMoin/auth/__init__.py +++ b/MoinMoin/auth/__init__.py @@ -138,6 +138,8 @@ from MoinMoin import log logging = log.getLogger(__name__) from MoinMoin import user, wikiutil +from urllib import quote, quote_plus +from werkzeug import abort, redirect def get_multistage_continuation_url(request, auth_name, extra_fields={}): }}} = Discussion = Could you please try with: {{{ from werkzeug import abort, redirect, url_quote, url_quote_plus ... (also change code below to use those calls) }}} Werkzeug's stuff is usually more comfortable and mostly (not ever) compatible with the stdlib stuff. Seems to work. -- JohannesBerg <> = Plan = ## This part is for Moin``Moin developers: * Priority: * Assigned to: * Status: fixed by http://hg.moinmo.in/moin/1.9/rev/105451cabedb ---- ## If you are a moin core developer, replace the category to Category* in these cases: ## Category MoinMoinNoBug - if this is not a bug. ## Category MoinMoinBugConfirmed - if you can confirm the bug on current code. ## Category MoinMoinBugFixed - after the bug is fixed in current code. CategoryMoinMoinBugFixed