Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UUID generation example looks wrong #470

Closed
vblazhkun opened this issue Dec 23, 2021 · 2 comments
Closed

UUID generation example looks wrong #470

vblazhkun opened this issue Dec 23, 2021 · 2 comments

Comments

@vblazhkun
Copy link

In the given text the UUID generation example of:

  • UUID Version: 5
  • Namespace UUID: {2BDE4A90-D05F-401C-9492-E40884EAD1D8}
  • Text: Ubuntu

Leads to UUID of {2C4DE342-38B7-51CF-B940-2309A097F518} which does not match to what we have with the standard RFC4122 implementations:

$ uuid -v 5 '2BDE4A90-D05F-401C-9492-E40884EAD1D8' 'Ubuntu'
8fa9feba-8e6e-5128-a6ba-e0926db2a4b4

$ ipython
IPython 7.30.1 -- An enhanced Interactive Python. Type '?' for help.
                                                                    
In [1]: import uuid                                                 
                                                                    
In [2]: u = uuid.UUID('{2BDE4A90-D05F-401C-9492-E40884EAD1D8}')     
                                                                    
In [3]: u                                                           
Out[3]: UUID('2bde4a90-d05f-401c-9492-e40884ead1d8')                
                                                                   
In [4]: uuid.uuid5(u, 'Ubuntu')                                     
Out[4]: UUID('8fa9feba-8e6e-5128-a6ba-e0926db2a4b4')

Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@Okeanos
Copy link
Contributor

Okeanos commented Jan 4, 2022

I think this is a duplicate of #320 (to be solved by #460).

@mattwojo
Copy link
Collaborator

Just merged the fix: #460

Thanks @vblazhkun (and thanks @Okeanos for the fix!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants