Re: [Cbor] Benjamin Kaduk's Yes on draft-ietf-cbor-sequence-01: (with COMMENT)

[Benjamin Kaduk <[e-mail]>]

On Wed, Sep 25, 2019 at 02:21:26PM +0200, Carsten Bormann wrote:
> On Sep 19, 2019, at 04:40, Benjamin Kaduk <[e-mail]> wrote:
> > 
> > On Wed, Sep 18, 2019 at 07:27:29PM -0700, Jim Schaad wrote:
> >> One response down below.
> >> 
> >> -----Original Message-----
> >> From: Benjamin Kaduk via Datatracker <[e-mail]> 
> >> Sent: Wednesday, September 18, 2019 4:41 PM
> >> To: The IESG <[e-mail]>
> >> Cc: [e-mail]; Jim Schaad <[e-mail]>; [e-mail]; [e-mail]; [e-mail]
> >> Subject: Benjamin Kaduk's Yes on draft-ietf-cbor-sequence-01: (with COMMENT)
> >> 
> >>> Section 5
> >> 
> >>> I might note that when COSE is applied to the elements of a sequence, the cryptographic protection is on a per-element basis, and thus there is no guarantee of relationship between level of protection, source authentication, time of generation, etc., across members of the sequence.
> >> 
> >> [JLS] I don't believe that there is any requirement that there be only a single element of a sequence inside of a COSE wrapper.  If three elements are generated at the same time then all three could be placed into a single encryption.  Thus I am not sure how to say that the protection is on a per-element basis.  
> > 
> > Perhaps (presumably?) this is just a question of semantics, but from where
> > I sit, the output of a COSE operation is a CBOR array (that has at least
> > three elements and maybe more depending on context, but that's not really
> > relevant for the point I want to make).  If I'm using COSE to protect
> > (some) elements of a CBOR sequence, then what goes into the sequence is the
> > CBOR array resulting from the COSE operation.  Undoing that COSE operation
> > may or may not produce more than one CBOR object, but those objects are not
> > elements of the CBOR sequence in question; they are contained within an
> > element of the CBOR sequence in question.  So, to me, the elements of the
> > sequence are necessarily a single COSE output, even if the consuming
> > application may assign different semantics to them.
> > 
> >> While it is true there is not necessarily a relationship between each COSE object, each COSE object does have the ability to provide that information for each element of the sequence.  You have more than what you started with, where you had none of that, and now you have it for each element.  You just need to do key validation on each decryption.  That might be a better way to say things in that a stream could have multiple sources and thus multiple keys.
> > 
> > Sure, making positive statements about what can be done is preferrable to
> > just making statements about what is not guaranteed (though perhaps both
> > would be needed; I haven't thought about what the text would look like very
> > much).
> 
> I have tried to address this in https://github.com/cbor-wg/seq/commit/0dcd3cf9e442a793a30a2093c6e42260f0fc6d80

I think that works; thanks.
Just to confirm my understanding: since COSE wraps the plaintext content in
a bstr, there's no change needed to allow for the sequence to be present?

-Ben