Minimal changes for GraalVM compatibility

zelark / nano-id

A unique string ID generator for Clojure and ClojureScript (・_・)ノ

MIT License

191 stars 9 forks source link

Minimal changes for GraalVM compatibility #15

Closed FieryCod closed 7 months ago

FieryCod commented 2 years ago

Hi @zelark! Thank you for the nano-id. I found that the current implementation is not compatible with GraalVM, because of extensive use of SecureRandom in static fields.

I found in benchmarks a little slowdown, but please confirm it on your side.

zelark commented 2 years ago

Hi @FieryCod, thanks for your contribution. I think it's a good point to have GraalVM compatibility. I'll check it later.

FieryCod commented 2 years ago

Sure thing. Feel free to ping me when necessary 😀

coreagile commented 1 year ago

When do folks think this can get merged in?