-
Notifications
You must be signed in to change notification settings - Fork 867
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
fix: Upgrade go-retryablehttp to v0.7.7 #3743
base: master
Are you sure you want to change the base?
Conversation
edfaf08
to
a85feb7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3743 +/- ##
=======================================
Coverage 83.87% 83.87%
=======================================
Files 163 163
Lines 18564 18564
=======================================
Hits 15571 15571
Misses 2120 2120
Partials 873 873 ☔ View full report in Codecov by Sentry. |
Go Published Test Results2 171 tests 2 171 ✅ 2m 54s ⏱️ Results for commit a85feb7. |
E2E Tests Published Test Results 4 files 4 suites 3h 31m 8s ⏱️ For more details on these failures, see this check. Results for commit a85feb7. |
a85feb7
to
6054058
Compare
Quality Gate passedIssues Measures |
Testing Published Test Results 5 files 123 suites 3h 33m 26s ⏱️ For more details on these failures, see this check. Results for commit 6054058. |
6054058
to
5a34af8
Compare
Published E2E Test Results 4 files 4 suites 3h 17m 27s ⏱️ For more details on these failures, see this check. Results for commit 796a467. ♻️ This comment has been updated with latest results. |
Published Unit Test Results2 275 tests 2 275 ✅ 2m 59s ⏱️ Results for commit 796a467. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM:
- Confirmed that all the dependency updates in
go.mod
are directly related to the upgrade ofgo-retryablehttp
to the requisite version to fix the CVE - Unit tests and E2E tests are passing, no obvious issues in 'E2E test results' logs
Signed-off-by: Rizwana777 <[email protected]>
5a34af8
to
796a467
Compare
Quality Gate passedIssues Measures |
Hi @zachaller can you please review and merge this PR? |
fixes #3700