-
Notifications
You must be signed in to change notification settings - Fork 215
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
Dependency update to fix CVEs #464
Conversation
Could you please rebase on top of master, as I just merged #462? Thanks! |
I will do the rebasing. |
PR was rebased. I could benefit from a test run to see if things comply. |
@kamilogorek can you have another look please ? |
@TheoBrigitte the code is alright and the tests are passing. However, we are in the middle of a HackWeek at Sentry, and we've code frozen until the end of the week. Will merge it on Monday :) |
Ok, thanks for the feedback :) |
@kamilogorek any chance this gets released? |
Towards: #465
On latest
master
, nancy found 4 vulnerable dependencies with 5 high to critical CVEs.This PR updates dependencies to get rid of those CVEs.
Direct dependencies update (manually triggered) :
Notable indirect dependencies update (done automatically by go) :