darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-02 17:31:14 -05:00
Code Issues Packages Projects Releases Wiki Activity

change TypeError test

Browse Source
This commit is contained in:
Kenneth Kehl
2024-09-04 13:19:50 -07:00
parent cf246aba7d
commit 33568d431f
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/authentication/auth.py
View File

@@ -67,8 +67,10 @@ def requires_internal_auth(expected_client_id):
f"TODO REMOVE: Enter requires_internal_auth with expected client id {expected_client_id}" f"TODO REMOVE: Enter requires_internal_auth with expected client id {expected_client_id}"
) )
# Looks like we are hitting this for some reason # Looks like we are hitting this for some reason
if expected_client_id not in current_app.config.get("INTERNAL_CLIENT_API_KEYS"): # expected_client_id looks like ADMIN_CLIENT_USERNAME on the admin side, and
# INTERNAL_CLIENT_API_KEYS is a dict
keys = current_app.config.get("INTERNAL_CLIENT_API_KEYS") keys = current_app.config.get("INTERNAL_CLIENT_API_KEYS")
if keys.get(expected_client_id) is None:
debug_not_production( debug_not_production(
f"TODO REMOVE: {expected_client_id} not in {keys}, raising TypeError\n" f"TODO REMOVE: {expected_client_id} not in {keys}, raising TypeError\n"
) )