Skip to content

Commit 72d1ca1

Browse files
authored
Merge pull request #100 from kurusugawa-computer/fix-logger
logにauthorization headerを出力しないようにする
2 parents 5f9e241 + d909f1f commit 72d1ca1

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

annofabapi/utils.py

+7-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
44
"""
55

6+
import copy
67
import datetime
78
import logging
89
from pathlib import Path
@@ -44,9 +45,14 @@ def log_error_response(arg_logger: logging.Logger, response: requests.Response):
4445
"""
4546

4647
if 400 <= response.status_code < 600:
48+
headers = copy.deepcopy(response.request.headers)
49+
if "Authorization" in headers:
50+
# logにAuthorizationを出力しないようにマスクする
51+
headers["Authorization"] = "***"
52+
4753
arg_logger.debug(f"status_code = %s, response.text = %s", response.status_code, response.text)
4854
arg_logger.debug(f"request.url = %s %s", response.request.method, response.request.url)
49-
arg_logger.debug("request.headers = %s", response.request.headers)
55+
arg_logger.debug("request.headers = %s", headers)
5056
arg_logger.debug("request.body = %s", response.request.body)
5157

5258

0 commit comments

Comments
 (0)