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.

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

Update dependencies and silence errors #1370

Merged
merged 1 commit into from
Feb 12, 2024
Merged

Update dependencies and silence errors #1370

merged 1 commit into from
Feb 12, 2024

Conversation

Swatinem
Copy link
Member

@Swatinem Swatinem commented Feb 12, 2024

Since basically forever, rust-minidump has been spewing unactionable warnings related to XSTATE which are being logged as Sentry breadcrumbs.
Also, a ton of S3 related NoSuchBucket errors have been logged internally which should ideally have been handled silently, but it looks like the S3 SDK does not properly handle those errors itself.

#skip-changelog

Since basically forever, `rust-minidump` has been spewing unactionable warnings related to `XSTATE` which are being logged as Sentry breadcrumbs.
Also, a ton of S3 related `NoSuchBucket` errors have been logged internally which should ideally have been handled silently, but it looks like the S3 SDK does not properly handle those errors itself.
@Swatinem Swatinem requested a review from loewenheim February 12, 2024 11:22
@Swatinem Swatinem self-assigned this Feb 12, 2024
Copy link

codecov bot commented Feb 12, 2024

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (6094436) 76.11% compared to head (b77f7e2) 76.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1370      +/-   ##
==========================================
- Coverage   76.11%   76.09%   -0.03%     
==========================================
  Files         101      101              
  Lines       15258    15262       +4     
==========================================
- Hits        11614    11613       -1     
- Misses       3644     3649       +5     

@Swatinem Swatinem merged commit 34b3ed7 into master Feb 12, 2024
14 of 15 checks passed
@Swatinem Swatinem deleted the swatinem/upd branch February 12, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants