Remove uuid dependency in lib-core and lib-utils - #333
Conversation
|
Actually we don't need /hold |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #333 +/- ##
==========================================
+ Coverage 30.51% 30.91% +0.40%
==========================================
Files 75 76 +1
Lines 1973 1986 +13
Branches 348 348
==========================================
+ Hits 602 614 +12
- Misses 1342 1343 +1
Partials 29 29 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/hold cancel |
d2f5a07 to
df232d3
Compare
|
@vojtechszocs: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: logonoff, vojtechszocs The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Summary
Remove uuid dependency in lib-core and lib-utils packages.
uuidmethodv4is replaced withcrypto.getRandomValuesbased utility function.Motivation
uuidhas a known security vulnerabilityCVE-2026-41907(1) (2) which impacts itsv3,v5andv6API methods.Both lib-core and lib-utils packages currently use
uuidmethodv4only - not impacted by this CVE.It's best to remove
uuiddependency and use standard Crypto API to generate UUID v4 compatible values.