Skip to content

Commit

Permalink
fix: #1711 clear-text logging of sensitive information (#1712)
Browse files Browse the repository at this point in the history
  • Loading branch information
craigyu authored Dec 20, 2024
1 parent d12b183 commit 2890ec7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions server/admin_management/api/config/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ def get_db_string():
else:
db_conn_string = get_local_dev_db_string()

LOGGER.debug(f"Database connection url: {db_conn_string}")
return db_conn_string


Expand Down Expand Up @@ -170,4 +169,4 @@ def get_idim_proxy_api_key():

def get_gc_notify_email_api_key():
gc_notify_email_api_key = get_env_var("GC_NOTIFY_EMAIL_API_KEY")
return gc_notify_email_api_key
return gc_notify_email_api_key
3 changes: 1 addition & 2 deletions server/backend/api/config/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ def get_db_string():
else:
db_conn_string = get_local_dev_db_string()

# LOGGER.debug(f"Database connection url: {db_conn_string}")
return db_conn_string


Expand Down Expand Up @@ -195,4 +194,4 @@ def get_api_key_for_update_user_info():
return os.environ.get("FAM_UPDATE_USER_INFO_API_KEY")

def get_requester_name_for_update_user_info():
return os.environ.get("FAM_UPDATE_USER_INFO_REQUESTER_NAME") or 'CMENG'
return os.environ.get("FAM_UPDATE_USER_INFO_REQUESTER_NAME") or 'CMENG'

0 comments on commit 2890ec7

Please sign in to comment.