-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: update selfsigned dependency for a security patch #2752
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2752 +/- ##
=======================================
Coverage 93.77% 93.77%
=======================================
Files 34 34
Lines 1333 1333
Branches 381 381
=======================================
Hits 1250 1250
Misses 81 81
Partials 2 2 Continue to review full report at Codecov.
|
any news on this? Thank you :) |
nope.... local testing works fine, unittests only crashes on azure testing instances PS: sorry for the duplicated MR. I did not notice at first sight |
is there an anticipated timeframe for fixing failing checks and getting this merged? thank you. |
CVE function in question does not appear to be directly used in this repo or @hiroppy / @evilebottnawi / @Loonride thoughts on using yarn resolutions or the npm equivalent until this is merged? |
Duplicate #2740 |
For Bugs and Features; did you add new tests?
Motivation / Use-Case
Security vulnerability in the selfsigned dependency caused by node-forge
Breaking Changes
Additional Info