From d1cbf9d8b9d0353faece814d674dff1ff80d9393 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Oct 04 2016 15:53:14 +0000 Subject: Open a session to the DB for every message received and close it afterward This ensure the data isn't cached and that updating the settings of a project will be reflected in this service (thus if one updates the URL, the service will be updated). Fixes https://pagure.io/pagure/issue/1248 --- diff --git a/webhook-server/pagure-webhook-server.py b/webhook-server/pagure-webhook-server.py index 9c3c2f3..8697e0d 100644 --- a/webhook-server/pagure-webhook-server.py +++ b/webhook-server/pagure-webhook-server.py @@ -123,8 +123,10 @@ def handle_messages(): if '/' in projectname: namespace, projectname = projectname.split('/', 1) + session = pagure.lib.create_session(pagure.APP.config['DB_URL']) project = pagure.lib.get_project( - session=pagure.SESSION, name=projectname, user=username) + session=session, name=projectname, user=username) + session.close() log.info('Got the project, going to the webhooks') call_web_hooks(project, data['topic'], data['msg'])