Skip to content

Commit 2fdfae7

Browse files
authored
chore(logs): Remove logging for entire response object (#62123)
This log outputs the entire response payloads which can contain sensitive information like email addresses, commit messages, links, etc. It also makes generic searches through logs return a lot of noisy data
1 parent 0f1f443 commit 2fdfae7

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

src/sentry/integrations/github/blame.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from isodate import parse_datetime
1010

1111
from sentry.integrations.mixins.commit_context import CommitInfo, FileBlameInfo, SourceLineInfo
12-
from sentry.utils import json
1312

1413
logger = logging.getLogger("sentry.integrations.github")
1514

@@ -114,10 +113,7 @@ def extract_commits_from_blame_response(
114113
back to the correct file.
115114
"""
116115
file_blames: list[FileBlameInfo] = []
117-
logger.info(
118-
"get_blame_for_files.extract_commits_from_blame.missing_repository",
119-
extra={**extra, "response": json.dumps(response)},
120-
)
116+
logger.info("get_blame_for_files.extract_commits_from_blame.missing_repository", extra=extra)
121117
for repo_index, (full_repo_name, ref_mapping) in enumerate(file_path_mapping.items()):
122118
repo_mapping: Optional[GitHubRepositoryResponse] = response.get("data", {}).get(
123119
f"repository{repo_index}"

0 commit comments

Comments
 (0)