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 the value type 'bstr' to counter signature #52

Closed
fpalombini opened this issue Dec 21, 2015 · 3 comments
Closed

add the value type 'bstr' to counter signature #52

fpalombini opened this issue Dec 21, 2015 · 3 comments

Comments

@fpalombini
Copy link

We propose to give the choice between the whole COSE_Signature structure or only the bstr 'signature' as "counter signature" value type. In CDDL, that would give:

Generic_Headers = (
...
? 7 => COSE_Signature / bstr, ; Counter signature
...
)

on section 3.1 and in Table 2.
If we only send the btstr 'signature', we assume that the alg for the counter signature is also pre-established between nodes and is included in the "alg" parameter.

@jimsch
Copy link

jimsch commented Jan 21, 2016

Per discussion on the list, this has been changed from a request to modify the current existing counter signature attribute to defining a new one. The reason for the change in the request is that the input for the new counter signature computation process should reflect the ToBeSigned structure of the COSE_Sign1 rather than the full COSE_Sign structure. This better reflects the circumstances that it is being designed for.

@selfissued
Copy link

It's not clear to me why another way of doing this is being requested. Are both really needed?

@fpalombini
Copy link
Author

Hi Mike, the goal behind this is again to reduce the size of the COSE message: instead of using a COSE_Signature structure (which itself contains headers and bstr signature value), we would like to use the simple bstr signature value directly. This would reduce the size of 3B: 1 for the array, 1 for the unprotected header and 1 for the protected header.

(the discussion about this in the mailing list is [here]([COSE] Issue - add the value type 'bstr' to counter signature)).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants