From 61ac283c383c844b47a6124498e6b18b516a28eb Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jan 31 2018 12:13:10 +0000 Subject: Fix the order of the before_request functions Local authentication needs to call _check_session_cookie before the request is ran but that call requires a session to connect to the database with and that is done in set_request so instead of ensuring _check_session_cookie is always run first, make it be after set_request. Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/flask_app.py b/pagure/flask_app.py index 14e00a4..088bbfc 100644 --- a/pagure/flask_app.py +++ b/pagure/flask_app.py @@ -149,7 +149,7 @@ def create_app(config=None): # Only import the login controller if the app is set up for local login if pagure_config.get('PAGURE_AUTH', None) == 'local': import pagure.ui.login as login - app.before_request_funcs[None].insert(0, login._check_session_cookie) + app.before_request(login._check_session_cookie) app.after_request(login._send_session_cookie) if perfrepo: