Skip to content

Replace uuid.New()/uuid.NewString() with a safe alternative #125

@jay-dee7

Description

@jay-dee7

Describe the bug
We're using a lot of uuids and they're generated using a non-safe method. Both uuid.New() and uuid.NewString() can panic and a better way could be to use id, err := uuid.NewRandom()

To Reproduce
nil, this bug hasn't occurred so far

Expected behavior
creating uuids should never result in panics and if they panic, the error should be handled.

Screenshots
Screenshot 2022-03-28 at 1 24 25 AM

Log Files
nil

Desktop (please complete the following information):
nil
Additional context
nil

Metadata

Metadata

Assignees

Labels

bugSomething isn't workinggood first issueGood for newcomershelp wantedExtra attention is needed

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions