From c6f63140a27a517846ce34a9ceb3cf4a5f052203 Mon Sep 17 00:00:00 2001 From: Diana Huang Date: Tue, 7 May 2013 11:42:39 -0400 Subject: [PATCH] Hide the API key during logging. --- lms/lib/comment_client/utils.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lms/lib/comment_client/utils.py b/lms/lib/comment_client/utils.py index 860035dc06..53bdd462ad 100644 --- a/lms/lib/comment_client/utils.py +++ b/lms/lib/comment_client/utils.py @@ -37,6 +37,10 @@ def perform_request(method, url, data_or_params=None, *args, **kwargs): else: response = requests.request(method, url, params=data_or_params, timeout=5) except Exception as err: + # remove API key if it is in the params + if 'api_key' in data_or_params: + log.info('Deleting API key from params') + del data_or_params['api_key'] log.exception("Trying to call {method} on {url} with params {params}".format( method=method, url=url, params=data_or_params)) # Reraise with a single exception type