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

Replace package satori/go.uuid #257

Closed
josselin-c opened this Issue Jul 11, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@josselin-c

josselin-c commented Jul 11, 2018

This issue has been opened automaticaly because we think you are using satori/uuid in your code. Unfortunately, satori/uuid has a critical defect where it doesn't generate random UUIDV4s satori/go.uuid#73 and should be replaced as the maintainer has been unresponsive for 5 months with the bug still open. Alternative packages you should consider: https://github.com/google/uuid https://github.com/pborman/uuid

@nanohard

This comment has been minimized.

Show comment
Hide comment
@nanohard

nanohard Aug 17, 2018

Contributor

The most recent maintained version is:
github.com/gofrs/uuid

Contributor

nanohard commented Aug 17, 2018

The most recent maintained version is:
github.com/gofrs/uuid

nanohard added a commit to nanohard/ponzu that referenced this issue Aug 17, 2018

nanohard added a commit to nanohard/ponzu that referenced this issue Aug 17, 2018

nilslice added a commit that referenced this issue Aug 19, 2018

Replaced satori/go.uuid with gofrs/uuid. (#268)
* Changed files to use new uuid pkg.

* Replace satori/go.uuid with gofrs/uuid. Closes #257.

* Deleted satori/go.uuid folder. Closes #257.

@nanohard nanohard self-assigned this Aug 20, 2018

nilslice added a commit that referenced this issue Aug 20, 2018

Replaced satori/go.uuid with gofrs/uuid. (#268)
* Changed files to use new uuid pkg.

* Replace satori/go.uuid with gofrs/uuid. Closes #257.

* Deleted satori/go.uuid folder. Closes #257.

@nilslice nilslice closed this in 52226ad Aug 20, 2018

nilslice added a commit that referenced this issue Aug 20, 2018

Replaced satori/go.uuid with gofrs/uuid. (#268)
* Changed files to use new uuid pkg.

* Replace satori/go.uuid with gofrs/uuid. Closes #257.

* Deleted satori/go.uuid folder. Closes #257.

nilslice added a commit that referenced this issue Aug 20, 2018

Replaced satori/go.uuid with gofrs/uuid. (#268)
* Changed files to use new uuid pkg.

* Replace satori/go.uuid with gofrs/uuid. Closes #257.

* Deleted satori/go.uuid folder. Closes #257.
@nilslice

This comment has been minimized.

Show comment
Hide comment
@nilslice

nilslice Aug 20, 2018

Member

@josselin-c, thank you for pointing this out. As mentioned by @nanohard, gofrs/uuid looks like a well-maintained replacement for the original satori/uuid, and it has been updated.

Member

nilslice commented Aug 20, 2018

@josselin-c, thank you for pointing this out. As mentioned by @nanohard, gofrs/uuid looks like a well-maintained replacement for the original satori/uuid, and it has been updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment