Why doesn't UUID version 5 give expected results? about elixir-uuid

[homanchou]

As a work around for now, specifying the uuid string of the namespace does in fact work:

iex> UUID.uuid5("6ba7b812-9dad-11d1-80b4-00c04fd430c8", "hi.bye")
"e1cd0925-0852-5171-9e44-5fce3d17ee7d"

Here is a list of uid namespaces:

http://stackoverflow.com/questions/10867405/generating-v5-uuid-what-is-name-and-namespace

The UUID RFC pre-defines four namespaces for you:

NameSpace_DNS: {6ba7b810-9dad-11d1-80b4-00c04fd430c8}
NameSpace_URL: {6ba7b811-9dad-11d1-80b4-00c04fd430c8}
NameSpace_OID: {6ba7b812-9dad-11d1-80b4-00c04fd430c8}
NameSpace_X500:{6ba7b814-9dad-11d1-80b4-00c04fd430c8}

Using the atoms does not work, it looks like the issue might be in this line
https://github.com/zyro/elixir-uuid/blob/master/lib/uuid.ex#L330

Might be the wrong type of encoding compared to erlang version? https://github.com/avtobiff/erlang-uuid/blob/master/src/uuid.erl#L193

The nil namespace works, since it's all zeros. https://github.com/zyro/elixir-uuid/blob/master/lib/uuid.ex#L337

[zyro]

You're right, looks like it was a binary encoding issue. Thanks for the thorough investigation!

I've released an updated version: https://hex.pm/packages/uuid/1.1.6

Closed by #27.

[homanchou]

cool! Thanks for merging. I assume you're going to update the readme too? I think the uuid3 and uuid5 examples on it might be wrong too.