Skip to content

Add unique ID#11

Open
lenivene wants to merge 2 commits into
donjajo:masterfrom
lenivene:master
Open

Add unique ID#11
lenivene wants to merge 2 commits into
donjajo:masterfrom
lenivene:master

Conversation

@lenivene

Copy link
Copy Markdown

Generate a unique ID when insert value

Generate a unique ID when insert value
@kungknut

Copy link
Copy Markdown

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

donjajo commented Jan 25, 2020 via email

Copy link
Copy Markdown
Owner

@kungknut

Copy link
Copy Markdown

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

basteyy commented Mar 16, 2020

Copy link
Copy Markdown

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

aik099 commented Nov 30, 2022

Copy link
Copy Markdown

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