search

danakt / uuid-by-string

Generates the RFC-4122 Name-Based UUID
MIT License
66 stars 12 forks source link

RFC-4122 specifies using lowercase #1

Closed kanef closed 5 years ago

kanef commented 6 years ago

I'd like to point out that according to the spec,

The hexadecimal values “a” through “f” are output as lower case characters and are case insensitive on input.

So this tool is technically not quite compliant, although I don't know how much practical consequence that is since tools that read the id are required to ignore the case.

It's probably a one-line change here, but of course the documentation and tests would also need to change to match.

danakt commented 6 years ago

Thanks for your comment @kanef. I can't fix this in the current version, as this can lead to unknown consequences in projects where the package is used. I think that in the near future I'll make these changes in next major version