Skip to content

[Merged by Bors] - Update outdated dependencies#2425

Closed
AgeManning wants to merge 4 commits intounstablefrom
sec-updates
Closed

[Merged by Bors] - Update outdated dependencies#2425
AgeManning wants to merge 4 commits intounstablefrom
sec-updates

Conversation

@AgeManning
Copy link
Member

This updates some older dependencies to address a few cargo audit warnings.

The majority of warnings come from network dependencies which will be addressed in #2389.

This PR contains some minor dep updates that are not network related.

@michaelsproul
Copy link
Member

The Docker image failed to build -- I think it needs Rust 1.53.0 to be able to compile new libp2p

@michaelsproul
Copy link
Member

Some of the remote signer tests also broke, presumably because of the curl dep bump. Fixing them is usually a matter of replacing the strings. It's janky, but will be improved when the remote signer gets revamped.

@michaelsproul michaelsproul changed the title Security Updates Update outdated dependencies Jul 5, 2021
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@michaelsproul michaelsproul added the ready-for-merge This PR is ready to merge. label Jul 5, 2021
bors bot pushed a commit that referenced this pull request Jul 5, 2021
This updates some older dependencies to address a few cargo audit warnings.

The majority of warnings come from network dependencies which will be addressed in #2389. 

This PR contains some minor dep updates that are not network related.

Co-authored-by: Michael Sproul <[email protected]>
@bors
Copy link

bors bot commented Jul 5, 2021

@bors bors bot changed the title Update outdated dependencies [Merged by Bors] - Update outdated dependencies Jul 5, 2021
@bors bors bot closed this Jul 5, 2021
@michaelsproul michaelsproul deleted the sec-updates branch July 5, 2021 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge. security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants