fix: uuid collisions - Githubissues

amplitude / Amplitude-JavaScript

JavaScript SDK for Amplitude

MIT License

315 stars 132 forks source link

fix: uuid collisions #594

Closed Mercy811 closed 1 year ago

Mercy811 commented 1 year ago

Summary

https://amplitude.atlassian.net/browse/AMP-76984

Replace Math.random with crypto.getRandomValues to avoid uuid collisions

Testing:

Run npm test and pass
manually run the function and the result follows the uuid format required

Checklist

[ ] Does your PR title have the correct title format?
Does your PR have a breaking change?: