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
Replace node-uuid with uuid #12209
Replace node-uuid with uuid #12209
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔍 📣 💸
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - we're using v1()
and v4()
within Kibana and outputs appear consistent.
* replace node-uuid with uuid package * fix all the uuid call sites (cherry picked from commit 1ffa6df)
5.5/5.x: c32e765 |
Wait! I hadn't given my approval... |
You 💤 , you lose @simianhacker |
Yeah... except I was in the middle of figuring out why I was getting errors so I could check TSVB... Oh well. I guess we can fix it in QA |
* replace node-uuid with uuid package * fix all the uuid call sites
We generate a fair number of UUIDs in Kibana, and we've been using the wrong module for some time now.
node-uuid
was replaced withuuid
some time ago (years).This PR updates Kibana so it uses the right one. The major version changes in the package do not affect our use.
I noticed the warning when I was installing packages as part of #12025