Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Replace shortid with node-uuid #930

Closed
steida opened this issue Jun 2, 2016 · 5 comments · Fixed by #931
Closed

Replace shortid with node-uuid #930

steida opened this issue Jun 2, 2016 · 5 comments · Fixed by #931

Comments

@steida
Copy link
Contributor

steida commented Jun 2, 2016

dylang/shortid#70

I think it's just fine for public urls related to some domain, but it's no go for any security sensitive purpose. Should be highlighted somehow somewhere.

snewcomer added a commit to snewcomer/este that referenced this issue Jun 5, 2016
snewcomer added a commit to snewcomer/este that referenced this issue Jun 5, 2016
snewcomer added a commit to snewcomer/este that referenced this issue Jun 5, 2016
@steida
Copy link
Contributor Author

steida commented Jun 5, 2016

The problem with https://github.com/broofa/node-uuid is it seems to not be maintained anymore. shortid is just fine for urls.

@jgkim
Copy link
Contributor

jgkim commented Jun 5, 2016

CUID can be considered as an alternative to shortid.

@steida
Copy link
Contributor Author

steida commented Jun 5, 2016

Doesn't work in React Native paralleldrive/cuid#54

@SN4T14
Copy link

SN4T14 commented Jun 19, 2016

What about using defunctzombie's fork of node-uuid?

@steida
Copy link
Contributor Author

steida commented Jun 19, 2016

Must be something more supported to be included into Este :(

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

Successfully merging a pull request may close this issue.

3 participants