Skip to content

Commit 09c7b94

Browse files
committed
logging cleanup
1 parent cee1f39 commit 09c7b94

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

src/server/_common.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,6 @@ def before_request_execute():
9797
user = current_user
9898
api_key = resolve_auth_token()
9999

100-
# TODO: remove after testing -- what does `user` look like in logs? dont forget, we DONT wanna print `user` info to logs in prod!!!
101-
get_structured_logger("server_api").info("USER INFO", user=(user and user.as_dict))
102-
103-
# Log statement
104100
# TODO: replace this next call with: log_info_with_request("Received API request")
105101
get_structured_logger("server_api").info(
106102
"Received API request",
@@ -113,6 +109,7 @@ def before_request_execute():
113109
user_agent=request.user_agent.string,
114110
api_key=api_key,
115111
)
112+
116113
if not show_no_api_key_warning():
117114
if not _is_public_route() and api_key and not user:
118115
# if this is a privleged endpoint, and an api key was given but it does not look up to a user, raise exception:

0 commit comments

Comments
 (0)