login: add error handling for missing env variable

Signed-off-by: Vallari Agrawal <[email protected]>
ceph · Aug 31, 2023 · 315dbc7 · 315dbc7
1 parent 2deffd6
commit 315dbc7
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/routes/login.py b/src/routes/login.py
@@ -28,6 +28,8 @@ async def github_login():
     GET route for /login, (If first time) will redirect to github login page
     where you should authorize the app to gain access.
     """
+    if not GH_AUTHORIZATION_BASE_URL or not GH_CLIENT_ID:
+        return HTTPException(status_code=500, detail="Environment secrets are missing.")
     scope = "read:org"
     return RedirectResponse(
         f"{GH_AUTHORIZATION_BASE_URL}?client_id={GH_CLIENT_ID}&scope={scope}",