Upgrade ulid because of panics

goph / idgen

Opaque ID generators

MIT License

2 stars 0 forks source link

Upgrade ulid because of panics #5

Open thpk opened 5 years ago

thpk commented 5 years ago

I was testing out your modern go application, and left a container running for a few weeks (forgot to stop it). Tested again and I noticed some panics in the ulid generation. It probably has to do with math.Rand not supporting concurrent use?

See issue in underlying package https://github.com/oklog/ulid/issues/54

sagikazarmark commented 5 years ago

Thanks for reporting this issue, I'll take a look at it.