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 #70

Merged
merged 4 commits into from
Oct 14, 2021
Merged

update dependencies #70

merged 4 commits into from
Oct 14, 2021

Conversation

06kellyjac
Copy link
Contributor

@06kellyjac 06kellyjac commented Oct 13, 2021

Update dependencies to resolve tokio 1.6.1: RUSTSEC-2021-0072, patched: >=1.5.1, <1.6.0; >=1.6.3, <1.7.0; >=1.7.2, <1.8.0; >=1.8.1

NixOS/nixpkgs#141368

It might be worth adding dependabot if you don't want to remember intermittent dependency updates. Or at least set up the cargo audit GH action to run intermittently https://github.com/actions-rs/audit-check or if you don't like GH actions you could just trigger cargo-audit yourself.

I'm happy to add dependabot and the audit action in this PR if you're ok with that

@Johann150
Copy link
Collaborator

Johann150 commented Oct 13, 2021

I'm happy to add dependabot and the audit action in this PR if you're ok with that

yeah sure please go ahead

@06kellyjac
Copy link
Contributor Author

Badges, ci, and dependabot added.


Actually now that I look at it the vuln was updated out already.

The issue is a new release hasn't been tagged for 14 commits / 128 days

@Johann150 Johann150 merged commit 0d0c240 into mbrubeck:master Oct 14, 2021
@Johann150
Copy link
Collaborator

Thanks, I'll go ahead and release a new version.

@Johann150
Copy link
Collaborator

new version has been released to crates.io and with prebuilt binaries

@06kellyjac
Copy link
Contributor Author

Awesome, thanks!

I've got another PR open for the other updates (not fixing vulns) but there's some breaking changes I'm wrestling with around rusttls 😩

@Johann150
Copy link
Collaborator

yeah, tokio-rustls isn't updated yet it seems #72 (comment)

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