From 62fe1de12d4dd3b6fb3e48693e7e98ab617f7e07 Mon Sep 17 00:00:00 2001 From: Zohar Meir <33152084+zmeir@users.noreply.github.com> Date: Tue, 22 Aug 2023 18:28:06 +0300 Subject: [PATCH] Remove redundant toggle --- pr_agent/servers/github_app.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pr_agent/servers/github_app.py b/pr_agent/servers/github_app.py index 8bd0fd4d..a80daa03 100644 --- a/pr_agent/servers/github_app.py +++ b/pr_agent/servers/github_app.py @@ -51,12 +51,11 @@ async def get_body(request): except Exception as e: logging.error("Error parsing request body", e) raise HTTPException(status_code=400, detail="Error parsing request body") from e - if get_settings().github_app.verify_signature: + webhook_secret = getattr(get_settings().github, 'webhook_secret', None) + if webhook_secret: body_bytes = await request.body() signature_header = request.headers.get('x-hub-signature-256', None) - webhook_secret = getattr(get_settings().github, 'webhook_secret', None) - if webhook_secret: - verify_signature(body_bytes, webhook_secret, signature_header) + verify_signature(body_bytes, webhook_secret, signature_header) return body