From 9cdbca6f925649bf9748a31db1ee42d644d1a644 Mon Sep 17 00:00:00 2001 From: amercader Date: Tue, 6 Sep 2016 12:02:51 +0100 Subject: [PATCH] [#3229] Dispose of the SQLAlchemy session after each request This was also done on the Pylons BaseController, and it's necessary to avoid the dreaded DetachedInstanceError during tests --- ckan/config/middleware/flask_app.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/ckan/config/middleware/flask_app.py b/ckan/config/middleware/flask_app.py index 755146e9622..a8f243dffaf 100644 --- a/ckan/config/middleware/flask_app.py +++ b/ckan/config/middleware/flask_app.py @@ -20,6 +20,7 @@ from paste.deploy.converters import asbool from fanstatic import Fanstatic +import ckan.model as model from ckan.lib import helpers from ckan.lib import jinja_extensions from ckan.common import config, g, request, ungettext @@ -127,6 +128,10 @@ def ckan_before_request(): @app.after_request def ckan_after_request(response): + + # Dispose of the SQLALchemy session + model.Session.remove() + # Check session cookie response = check_session_cookie(response)