Skip to content

Conversation

@cce
Copy link
Contributor

@cce cce commented Apr 22, 2025

Summary

This upgrades some package versions (Go 1.23.9, golang.org/x/net, echo) and their dependencies, to satisfy vulnerability scanners. However, go-algorand doesn't seem to be using the vulnerable features of these software versions.

Test Plan

Existing tests should pass.

algorandskiy
algorandskiy previously approved these changes Apr 22, 2025
onetechnical
onetechnical previously approved these changes Apr 23, 2025
@algorandskiy
Copy link
Contributor

@cce mind resolving conflicts/redoing?

@cce cce dismissed stale reviews from onetechnical and algorandskiy via af87e18 May 30, 2025 01:56
@cce
Copy link
Contributor Author

cce commented May 30, 2025

Updated

@codecov
Copy link

codecov bot commented May 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.94%. Comparing base (2dbbf58) to head (ee711c9).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6307      +/-   ##
==========================================
- Coverage   51.95%   51.94%   -0.01%     
==========================================
  Files         653      653              
  Lines       87461    87461              
==========================================
- Hits        45439    45432       -7     
- Misses      39147    39151       +4     
- Partials     2875     2878       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@algorandskiy algorandskiy requested a review from gmalouf May 30, 2025 16:46
@gmalouf gmalouf merged commit 3fd5a70 into algorand:master May 30, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants