From 39cc176ba9d1a5a495c4755e2396273ebba3d2d1 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 28 2017 12:54:05 +0000 Subject: Have the web-hook use the signed content and have a content-type header --- diff --git a/webhook-server/pagure-webhook-server.py b/webhook-server/pagure-webhook-server.py index cd10e40..2b35227 100644 --- a/webhook-server/pagure-webhook-server.py +++ b/webhook-server/pagure-webhook-server.py @@ -81,8 +81,8 @@ def call_web_hooks(project, topic, msg, urls): 'X-Pagure-Signature': hashhex, 'X-Pagure-Signature-256': hashhex256, 'X-Pagure-Topic': topic, + 'Content-Type': 'application/json', } - msg = json.dumps(msg) for url in urls: url = url.strip() log.info('Calling url %s' % url) @@ -90,7 +90,7 @@ def call_web_hooks(project, topic, msg, urls): req = requests.post( url, headers=headers, - data={'payload': msg}, + data=content, timeout=60, ) if not req: