Skip to content
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

Add unique ID #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add unique ID #11

wants to merge 2 commits into from

Conversation

lenivene
Copy link

Generate a unique ID when insert value

Generate a unique ID when insert value
@kungknut
Copy link

According to the official php documentation at php.net the function does not guarantee uniqueness.

I suppose this won't be a problem when running in development and demonstration environment but it's a limitation worth knowing.

@donjajo
Copy link
Owner

donjajo commented Jan 25, 2020 via email

@kungknut
Copy link

What's your thoughts about adding dependencies and using UUIDv4?

If you can accept adding ramsey/uuid as a dependency you could do something like this to generate a UUIDv4 string instead of uniqid();

Uuid::uuid4()->toString()

@basteyy
Copy link

basteyy commented Mar 16, 2020

This is why I have not merged the pull yet. Finding a right approach, haven't gotten enough time to think it through

On Sat, 25 Jan 2020, 10:12 Philip Lundqvist, @.***> wrote: According to the official php documentation at php.net https://www.php.net/manual/en/function.uniqid.php#refsect1-function.uniqid-returnvalues the function does not guarantee uniqueness. I suppose this won't be a problem when running in development and demonstration environment but it's a limitation worth knowing. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#11?email_source=notifications&email_token=AC4YZGDE4M54VF7FWTZTM3DQ7P665A5CNFSM4JCXTCC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ4YRMY#issuecomment-578390195>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC4YZGCZBXEDANU27RQD7TTQ7P665ANCNFSM4JCXTCCQ .

What about using the function with time and options more_entropy: uniqid(time(), true);.

@aik099
Copy link

aik099 commented Nov 30, 2022

Or you can include paragonie/random_compat as a dependency and use random_int and such PHP7 functions even on PHP5 to get unique random values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants