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

Add COSE_Key support #147

Closed
wants to merge 3 commits into from
Closed

Add COSE_Key support #147

wants to merge 3 commits into from

Conversation

setrofim
Copy link
Contributor

@setrofim setrofim commented Jun 7, 2023

Add support for marshalling/unmarshalling COSE_Key structures, and using them for signing and verification. At the moment, only RFC8152 is implemented (OKP and EC2 signatures); RFC8230 (RSASSA-PSS signatures) is NOT implemented.

A number of elements in RFC8152 can be represented as either integer
values or string names. This type will allow uniform handling of such
elements when (un)marshalling to/from CBOR.

Signed-off-by: Sergei Trofimov <sergei.trofimov@arm.com>
RFC8152 specifies that an algorithm may be identified either by its
integer value, or by its string name. This adds support for latter.

Signed-off-by: Sergei Trofimov <sergei.trofimov@arm.com>
Add support for COSE_Key structure, and associated enums for key type,
ECC curve, and key ops, as defined in RFC8152.

Signed-off-by: Sergei Trofimov <sergei.trofimov@arm.com>
@setrofim
Copy link
Contributor Author

setrofim commented Jun 7, 2023

I have made the mistake of using the GitHub's "commit suggestion" feature while at the same time making commits for non-suggestion comments on my local version of the brach. While rebasing and squashing everything worked fine on the brach, GitHub seems to have gotten confused, and was refusing to update the old pull request to the new tip of sertrofim brach.

To get around that, I created a new branch and submitted this pull request. Weridly doing so seems to have "unstuck" the original pull request, renderring this one unnecessary. Closing this now. going back to #146. Apologies for the churn.

@setrofim setrofim closed this Jun 7, 2023
@setrofim setrofim deleted the cose-key branch June 8, 2023 07:42
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 this pull request may close these issues.

1 participant