From b21cfb53f34dc8d224196c4a0e1f4b4586827a2d Mon Sep 17 00:00:00 2001 From: jvanbraekel <29703119+jvanbraekel@users.noreply.github.com> Date: Thu, 21 Feb 2019 15:17:40 +0100 Subject: [PATCH] Fix redirect url when login failed (invalid login) Hello, This is broken since 1457e12f702d203c1c6b4eb731f7edb3057bfaf2 . 'self.url' does'nt have any get_vars,post_vars, only one parameter called vars. I suggest to pass all variables instead with request.vars that will encompass both get and post vars... Another possibility would be to change the self.url signature but I don't see the purpose of making the distinction between post and get vars in this case. --- gluon/tools.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/gluon/tools.py b/gluon/tools.py index e1c20c82..f3b4e28b 100644 --- a/gluon/tools.py +++ b/gluon/tools.py @@ -2640,9 +2640,7 @@ class Auth(AuthAPI): callback(onfail, None) if 'password' in request.post_vars: del request.post_vars['password'] - redirect( - self.url(args=request.args, get_vars=request.get_vars, post_vars=request.post_vars), - client_side=settings.client_side) + redirect(self.url(args=request.args, vars=request.vars),client_side=settings.client_side) else: # use a central authentication server cas = settings.login_form