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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for ordered UUID's #7

Closed
wants to merge 1 commit into from

Conversation

wouterdijkstra
Copy link

Added a little config option to use ordered UUID's instead of UUIDv4's.
Added a link to an article about (security) differences/advantages in the README as well.

--
This is a first for me, hope I'm doing things right! 馃槵

@freekmurze
Copy link
Member

I'd like to keep this package very simple.

Would there be any downsides of using ordered uuids by default?

@wouterdijkstra
Copy link
Author

First: There's the security argument; covered here.

It'd conclude the difference is negligible.

Second; ordered UUID's theoretically 'expose' the timestamp the record was created. This also means they're sortable by default, though.

Personally, for the goal of this package I'd say there are no downsides.

@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Jul 12, 2021
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.

None yet

3 participants