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.

Sign up for GitHub

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

Jump to bottom

crypto/rsa: EncryptOAEP documentation says "if a given public key is used to decrypt" instead of "encrypt" #44777

Closed
kocmo opened this issue Mar 4, 2021 · 3 comments
Closed

crypto/rsa: EncryptOAEP documentation says "if a given public key is used to decrypt" instead of "encrypt" #44777

kocmo opened this issue Mar 4, 2021 · 3 comments
Labels
Documentation Issues describing a change to documentation. FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Backlog

Comments

@kocmo
Copy link

kocmo commented Mar 4, 2021

What did you do?

Opened https://golang.org/pkg/crypto/rsa/#EncryptOAEP in browser

What did you expect to see?

For example, if a given public key is used to encrypt two types of messages, then distinct label values could be used to ensure that a ciphertext for one purpose cannot be used for another by an attacker.

What did you see instead?

For example, if a given public key is used to decrypt two types of messages then distinct label values could be used to ensure that a ciphertext for one purpose cannot be used for another by an attacker.
@gopherbot gopherbot added this to the Unreleased milestone Mar 4, 2021
@gopherbot gopherbot added the Documentation Issues describing a change to documentation. label Mar 4, 2021
@dmitshur dmitshur changed the title x/website: documentation error at https://golang.org/pkg/crypto/rsa/#EncryptOAEP crypto/rsa: EncryptOAEP documentation says "if a given public key is used to decrypt" instead of "encrypt" Mar 5, 2021
@dmitshur dmitshur modified the milestones: Unreleased, Backlog Mar 5, 2021
@dmitshur dmitshur added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Mar 5, 2021
@dmitshur
Copy link
Contributor

dmitshur commented Mar 5, 2021

CC @FiloSottile, @katiehockman, @rolandshoemaker via owners.

@katiehockman
Copy link
Contributor

I agree with @kocmo that the documentation is incorrect. If you are interested in fixing this, feel free to send a PR. Otherwise, one of us can look into it. Thanks.

@katiehockman katiehockman added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Mar 8, 2021
@rpickz rpickz mentioned this issue Mar 15, 2021
Closed
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/301714 mentions this issue: crypto/rsa: correct EncryptOAEP doc comment

@golang golang locked and limited conversation to collaborators Mar 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Documentation Issues describing a change to documentation. FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

crypto/rsa: correct EncryptOAEP doc comment rpickz/go
4 participants
@dmitshur @kocmo @gopherbot @katiehockman