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

Encoding of Name with only one attribute CommonName #140

Closed
xipki opened this issue Nov 29, 2023 · 4 comments
Closed

Encoding of Name with only one attribute CommonName #140

xipki opened this issue Nov 29, 2023 · 4 comments

Comments

@xipki
Copy link
Contributor

xipki commented Nov 29, 2023

In Section 3.1: If the text string contains only the symbols '0'–'9' or 'a'–'f', it is encoded as a CBOR byte string, prefixed with an initial byte set to '00'.

My questions are:

  1. What is about the upper-case hex letters A-F Shall common name with (only upper-case / mixed lower-case and upper-case) hex letters also be encoded?
  2. What if the text string have odd number of letters? If a zero is prefixed, then how to know whether it starts with 0X or X?
@highlunder
Copy link
Collaborator

The proposed formatting has been added after specific requests to accommodate (well-behaved) byte encoding of hex strings in the CN field. This means that it is up to the user and certificate creator to format the CN accordingly -- using only lowercase, and using an even number of letters, if they want to benefit from the more compact byte encoding

@xipki
Copy link
Contributor Author

xipki commented Nov 30, 2023

That is fine. But you need to add the limitation (even number of letters) to the text.

@xipki
Copy link
Contributor Author

xipki commented Nov 30, 2023

I cannot re-open this issue.
@highlunder Can you re-open this issue? I will appreciate if you can let an answered issue open for comments, and let the issue creator to resolve the issue.

@gselander gselander reopened this Nov 30, 2023
@gselander
Copy link
Collaborator

That is fine. But you need to add the limitation (even number of letters) to the text.

This is #135.

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