Skip to content

Commit

Permalink
Disable capture_message for tags until we cover more (#15652)
Browse files Browse the repository at this point in the history
  • Loading branch information
di authored Mar 21, 2024
1 parent 2774c1a commit 32df0fe
Showing 1 changed file with 10 additions and 7 deletions.
17 changes: 10 additions & 7 deletions warehouse/utils/wheel.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@
import re

import packaging.utils
import sentry_sdk

# import sentry_sdk

_PLATFORMS = [
(re.compile(r"^win_(.*?)$"), lambda m: f"Windows {_normalize_arch(m.group(1))}"),
Expand Down Expand Up @@ -93,11 +94,13 @@ def filename_to_pretty_tags(filename: str) -> list[str]:
version = _format_version(tag.interpreter.removeprefix("cp"))
pretty_tags.add(f"CPython {version}")
else:
with sentry_sdk.push_scope() as scope:
scope.fingerprint = [str(tag)]
sentry_sdk.capture_message(
f"wheel has unrecognized interpreter tag: {tag}. "
f"Filename: {filename}."
)
pass
# Disable until we can cover ~all tags
# with sentry_sdk.push_scope() as scope:
# scope.fingerprint = [str(tag)]
# sentry_sdk.capture_message(
# f"wheel has unrecognized interpreter tag: {tag}. "
# f"Filename: {filename}."
# )

return sorted(pretty_tags)

0 comments on commit 32df0fe

Please sign in to comment.