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

Deterministic Encoding of an Attribute #198

Closed
xipki opened this issue May 5, 2024 · 3 comments
Closed

Deterministic Encoding of an Attribute #198

xipki opened this issue May 5, 2024 · 3 comments
Assignees

Comments

@xipki
Copy link
Contributor

xipki commented May 5, 2024

In draft -09, an Attribute is encoded as follows

    Attribute = ( attributeType: int, attributeValue: text ) //
                ( attributeType: ~oid/pen, attributeValue: bytes )
  1. It is not clear how to encode the attribute, if it has the assigned int type, and
    • For emailAddress and domainComponent: the attribute value is not IA5String encoded,
    • For other attributes, the value is neither PRINTABLE STRING nor UTF8 STRING encoded.
  2. Shall the attribte type be negative (for PRINTABLE STRING) or posive (for UTF8 STRING), if the attribute type is of fixed string type, but not one of PRINTABLE STRING and UTF8 STRING, e.g. the emailAddress and domainComponent?
@highlunder
Copy link
Collaborator

  1. We need to clarify the text to remove the ambiguity, that we don't support
  2. Adapt the proposal given

@highlunder
Copy link
Collaborator

@xipki is welcome to push a PR with a clarifying proposal!

xipki added a commit to xipki/CBOR-certificates that referenced this issue Jul 6, 2024
highlunder added a commit that referenced this issue Jul 8, 2024
Deterministic Encoding of an Attribute, #198: clarifying text on allowed string types
@highlunder
Copy link
Collaborator

Accepting and merging PR with clarifying text!

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

2 participants