# HG changeset patch # User Søren Løvborg # Date 1437911930 -7200 # Node ID 8b8f51f36542f6ea096e5401d4fb6fbe2324d0d4 # Parent 071777309be11e4ad017f43f1e6188289f113b44 auth: actually use _determine_auth_user argument Fix silly mistake which slipped through the review. (We should not look up the session cookie again, when it's passed as a function argument.) diff -r 071777309be1 -r 8b8f51f36542 kallithea/lib/base.py --- a/kallithea/lib/base.py Fri Jul 24 17:53:20 2015 +0200 +++ b/kallithea/lib/base.py Sun Jul 26 13:58:50 2015 +0200 @@ -388,12 +388,11 @@ return AuthUser(api_key=api_key, is_external_auth=True) # Authenticate by session cookie - cookie = session.get('authuser') # In ancient login sessions, 'authuser' may not be a dict. # In that case, the user will have to log in again. - if isinstance(cookie, dict): + if isinstance(session_authuser, dict): try: - return AuthUser.from_cookie(cookie) + return AuthUser.from_cookie(session_authuser) except UserCreationError as e: # container auth or other auth functions that create users on # the fly can throw UserCreationError to signal issues with