-
Notifications
You must be signed in to change notification settings - Fork 121
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
Hash collision generating the UUID4 #476
Comments
The posted code uses That said, it's sufficient for versions 7.0+ to replace calls to Would recommend using Ramsey UUID first and see if that helps. If it doesn't, or if it's slow, then other approaches could be taken. |
Customer (my co) was using 7.2 at the time iirc, now 7.3, in transition to 7.4 |
Thanks, @passcod. With that info, we can probably eliminate issues with the Mersenne Twister implementation in the PHP engine. While it's a long shot to say it'll be reproducible, are you able to provide some sample code that demonstrates how you were able to get duplicate UUID? For example, tight loops? High concurrency? Also, what OS? |
Just normal use via rollbar error logging. No particular high usage (we hit a few thousand error logs a day at most), it was fairly random. We haven't had occurrences since early this year as we changed to generate the uuids our side (with ramsey's library) instead of relying on rollbar's internals.Servers are Ubuntu 18.04 LTS with a 5.3 kernel (using the "hwe" kernel package), with ondrej's PHP builds, running on a VMware cluster, if that matters any. ---- On Fri, 06 Nov 2020 12:54:41 +1300 [email protected] wrote ----
Thanks, @passcod. With that info, we can probably eliminate issues with the Mersenne Twister implementation in the PHP engine. While it's a long shot to say it'll be reproducible, are you able to provide some sample code that demonstrates how you were able to get duplicate UUID? For example, tight loops? High concurrency? Also, what OS?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
|
It looks like we are using a method to generate the UUID4 via this code - http://www.php.net/manual/en/function.uniqid.php#94959. This has caused UUID hash collision multiple times to the same customer. We need to look into a more standard way to generate the UUID4.
The text was updated successfully, but these errors were encountered: