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

use COSE_CertHash-style digest description in lieu of hash-entry #54

Closed
thomas-fossati opened this issue Feb 23, 2023 · 1 comment · Fixed by #56
Closed

use COSE_CertHash-style digest description in lieu of hash-entry #54

thomas-fossati opened this issue Feb 23, 2023 · 1 comment · Fixed by #56

Comments

@thomas-fossati
Copy link
Collaborator

We currently import hash-entry from CoSWID:

hash-entry = [
  hash-alg-id: int
  hash-value: bytes
]

which forces hash algorithm identifiers to be integer codes from IANA's named-information registry.

That's great, but imposes a dependency on yet another registry.

We could take instead the more merciful approach taken by RFC9360:

COSE_CertHash = [ hashAlg: (int / tstr), hashValue: bstr ]

which allows, for alg-ids, strings as an alternative to code points.

@nedmsmith
Copy link
Collaborator

I'm not passionate about it either way.

thomas-fossati added a commit that referenced this issue Feb 24, 2023
Fix #54

Signed-off-by: Thomas Fossati <thomas.fossati@arm.com>
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

Successfully merging a pull request may close this issue.

2 participants