Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix sentry scrubber and logs #7967

Merged
merged 2 commits into from
Apr 12, 2024
Merged

Fix sentry scrubber and logs #7967

merged 2 commits into from
Apr 12, 2024

Conversation

kozlovsky
Copy link
Collaborator

@kozlovsky kozlovsky commented Apr 12, 2024 •
edited
Loading

This PR fixes #7966.

  • The first commit reconfigures Sentry logs to be consistent with Tribler logs and allows seeing Sentry errors.
  • The second commit stops making substitutes to empty substrings in the middle of other strings.

@kozlovsky kozlovsky requested a review from a team as a code owner April 12, 2024 02:41
@kozlovsky kozlovsky requested review from egbertbouman and removed request for a team April 12, 2024 02:41
@kozlovsky kozlovsky force-pushed the fix/sentry_scrubber_and_logs branch 5 times, most recently from bbeb966 to 9a16979 Compare April 12, 2024 03:39
@kozlovsky kozlovsky force-pushed the fix/sentry_scrubber_and_logs branch from 9a16979 to 9dfe210 Compare April 12, 2024 06:21
@kozlovsky kozlovsky merged commit 03adf6a into Tribler:main Apr 12, 2024
20 checks passed
@kozlovsky kozlovsky deleted the fix/sentry_scrubber_and_logs branch April 12, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@egbertbouman egbertbouman egbertbouman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Reports to Sentry are silently ignored on some Tribler installations
2 participants
@kozlovsky @egbertbouman