From 1af28ddcbbdea7a65410dde04b9cf30baf312079 Mon Sep 17 00:00:00 2001 From: Leander Furumo Date: Mon, 15 Apr 2024 20:12:32 +0200 Subject: [PATCH] fix NOTIFY query asyncpg does not allow for arguments in NOTIFY queries, hence we have to use f-strings --- verification_server/src/server.py | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/verification_server/src/server.py b/verification_server/src/server.py index 51defdc..8ab750a 100644 --- a/verification_server/src/server.py +++ b/verification_server/src/server.py @@ -114,13 +114,9 @@ async def callback(request: Request, code: str, state: str): ) # Notify bot that user is verified - await app.state.pool.execute( - """ - NOTIFY galtinn_auth_complete, '$1 $2' - """, - discord_id, - user["sub"], # TODO: use UUID? - ) + # asyncpg does not allow for arguments in NOTIFY queries, hence we use f-strings + notify_query = f"NOTIFY galtinn_auth_complete, '{discord_id} {user['sub']}'" + await app.state.pool.execute(notify_query) return RedirectResponse(f"/success/{user['preferred_username']}", status_code=303)