Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-135244: improve wording ofuuid8 docs about CSPRNG#135433

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
picnixz merged 1 commit intopython:mainfromLamentXU123:main
Jun 13, 2025
Merged

gh-135244: improve wording ofuuid8 docs about CSPRNG#135433

picnixz merged 1 commit intopython:mainfromLamentXU123:main
Jun 13, 2025

Conversation

@LamentXU123
Copy link
Contributor

@LamentXU123LamentXU123 commentedJun 12, 2025•
edited by bedevere-appbot
Loading

This PR is a tiny fix of#135226

By default,a,b andc are generated by a non-cryptographically
secure pseudo-random number generator (CSPRNG). Use :func:uuid4 when
a UUID needs to be used in a security-sensitive context.

Well IMO, to avoid ambiguity, it may be changed to

By default,a,b andc are not generated by a cryptographically
secure pseudo-random number generator (CSPRNG). Use :func:uuid4 when
a UUID needs to be used in a security-sensitive context.

caz the term "CSPRNG" stands for "cryptographically secure pseudo-random number generator" not "non-cryptographically secure pseudo-random number generator".

Could be kind of dumb caz I'm not a English native speaker lol. But I think its a little weird about the original presentation.

I've mentioned this in a deleted comment of#135244, nvm. Skipping news and issue caz this is a tiny fix in doc. Thank u for ur time@picnixz.

📚 Documentation preview 📚:https://cpython-previews--135433.org.readthedocs.build/

@bedevere-appbedevere-appbot added awaiting review docsDocumentation in the Doc dir skip news labelsJun 12, 2025
@LamentXU123LamentXU123 changed the titleFix presentation in doc of uuid8 about CSPRNG#135244 Fix presentation in doc of uuid8 about CSPRNGJun 12, 2025
@LamentXU123LamentXU123 changed the title#135244 Fix presentation in doc of uuid8 about CSPRNGFix presentation in doc of uuid8 about CSPRNGJun 12, 2025
@picnixz
Copy link
Member

I'll review it this evening

LamentXU123 reacted with eyes emoji

@picnixzpicnixz changed the titleFix presentation in doc of uuid8 about CSPRNGgh-135244: improve wording ofuuid8 docs about CSPRNGJun 13, 2025
@picnixzpicnixz merged commit394d798 intopython:mainJun 13, 2025
45 checks passed
@github-project-automationgithub-project-automationbot moved this fromTodo toDone inDocs PRsJun 13, 2025
@miss-islington-app
Copy link

Thanks@LamentXU123 for the PR, and@picnixz for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestJun 13, 2025
…GH-135433)(cherry picked from commit394d798)Co-authored-by: LamentXU <108666168+LamentXU123@users.noreply.github.com>
@bedevere-app
Copy link

GH-135467 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelJun 13, 2025
@picnixz
Copy link
Member

picnixz commentedJun 13, 2025•
edited
Loading

We don't need a 3.13 backport because UUIDv8 has only been introduced in 3.14.

@LamentXU123
Copy link
ContributorAuthor

We don't need a 3.13 backport because UUIDv8 has only been introduced in 3.14.

sure. Nice work.

SeanDictionary reacted with thumbs up emojiSeanDictionary reacted with hooray emoji

picnixz pushed a commit that referenced this pull requestJun 13, 2025
…5433) (#135467)gh-135244: improve wording of `uuid8` docs about CSPRNG (GH-135433)(cherry picked from commit394d798)Co-authored-by: LamentXU <108666168+LamentXU123@users.noreply.github.com>
lkollar pushed a commit to lkollar/cpython that referenced this pull requestJun 19, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@picnixzpicnixzpicnixz approved these changes

Assignees

No one assigned

Labels

docsDocumentation in the Doc dirskip news

Projects

Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@LamentXU123@picnixz@ZeroIntensity
[8]ページ先頭
©2009-2025 Movatter.jp