Skip to content

Commit 8dd0396

Browse files
committed
fix: Fix authorization check for webhooks
1 parent f0301c9 commit 8dd0396

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

src/bot/web_server.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,14 @@ async def handle_request(self, request):
4545
4646
"""
4747
auth_header = request.headers.get("Authorization")
48-
if not auth_header or auth_header != os.getenv("HEALTHCHECKS_WEBHOOK_TOKEN"):
48+
49+
if not auth_header or not auth_header.startswith("Bearer "):
50+
return web.Response(text="Unauthorized", status=401)
51+
52+
token = auth_header.split("Bearer ")[1].strip()
53+
expected_token = os.getenv("HEALTHCHECKS_WEBHOOK_TOKEN")
54+
55+
if token != expected_token:
4956
return web.Response(text="Unauthorized", status=401)
5057

5158
data = await request.json()

0 commit comments

Comments
 (0)