-
Notifications
You must be signed in to change notification settings - Fork 945
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
deps(quic): upgrade quinn to address RUSTSEC-2023-0063 #4538
Conversation
This pull request has merge conflicts. Could you please resolve them @divagant-martian? 🙏 |
20d8998
to
484fc56
Compare
Thanks but if it is a patch-release, then users can upgrade that themselves! We should update our lockfile of course but we don't need to mention that in the changelog :) |
is your specific suggestion to simply remove the changelog @thomaseizinger? |
93220b4
to
d6b2f43
Compare
Yes please! :) |
not sure if you noticed but #4537 has workflows failing because of this, thus it needs to be included, not referring to users, so yeah. we are on the same page :) |
Ah yes, thank you for that! We don't require those for merging because there isn't always an immediate solution available but it is good to be fixed, thank you! :) |
Description
https://rustsec.org/advisories/RUSTSEC-2023-0063 was recently published, this address it.
Notes & open questions
na
Change checklist