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

CommonName Personal Name/Pseudonym options don't align with subject givenName/surname/pseudonym restriction #203

Closed
robplee opened this issue Jul 18, 2023 · 1 comment
Labels

Comments

@robplee
Copy link
Contributor

robplee commented Jul 18, 2023

Some extra text is needed to align commonName use with the restriction around subject givenName, surname and pseudonym.

If a subject:pseudonym is present, then subject:givenName and/or subject:surname should be absent and vice versa. However, if a subject:pseudonym is present, it's currently still possible to have a Personal Name in the commonName which seems at odds with the existing rule. Similarly, one could have a certificate containing a subject:givenName and subject:surname and then put one's pseudonym in the commonName despite the fact that there is a rule in place which would prevent the pseudonym being put into a subject:pseudonym field.

I don’t think it’s a difficult thing to fix though. Adding the following lines to 7.1.4.2.2(a) should close this hole effectively enough:

“If the subject:commonName contains a Pseudonym, then the subject:givenName and/or subject:surname attributes SHALL NOT be present.”

“If the subject:commonName contains a Personal Name, then the subject:pseudonym attribute SHALL NOT be present.”

srdavidson added a commit to srdavidson/smime that referenced this issue Sep 26, 2023
@srdavidson
Copy link
Contributor

See proposed change at srdavidson@4203e6d

srdavidson added a commit to srdavidson/smime that referenced this issue Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants