See inline
From: Neil Madden <[email protected]> Sent: Friday, September 20, 2019 8:09 AM To: Jim Schaad <[email protected]> Cc: ivaylo petrov <[email protected]>; [email protected]; [email protected] Subject: Re: [jose] π WGLC of draft-ietf-cose-webauthn-algorithms Thanks for the reply, comments in-line marked with [NEM]: On 20 Sep 2019, at 15:31, Jim Schaad <[email protected] <mailto:[email protected]> > wrote: From: jose < <mailto:[email protected]> [email protected]> On Behalf Of Neil Madden Sent: Friday, September 20, 2019 2:35 AM To: ivaylo petrov < <mailto:[email protected]> [email protected]> Cc: <mailto:[email protected]> [email protected]; <mailto:[email protected]> [email protected] Subject: Re: [jose] π WGLC of draft-ietf-cose-webauthn-algorithms Thanks, I wasn't aware of this draft. It looks ok, just a few comments from me: secp256k1 is mentioned in the context of signatures and the new ES256K JWS algorithm, but when it is registered in the JOSE Elliptic Curve registry it will also be usable for ECDH-ES encryption. The current draft mentions JOSE but only links to RFC 7515 (JWS). Is the intention that the curve be only used for signatures, or is it also intended for encryption? [JLS] That is an interesting question. Right now I would say that it is only for signatures, but it could be expanded to key agreement quite easily. Is there any need for it or are you just speculating? The big use I know of is bit coin which is only signatures and WebAuthn which is only signatures. [NEM] As soon as it is registered as a JOSE elliptic curve it can be used for ECDH-ES, so the draft should make a statement one way or another as to whether this is intended rather than standardizing that usage by side-effect IMO. I'm glad RS1 is not being registered for JOSE, although I'm still a bit surprised that it is being registered (even as deprecated) for a standard as new as COSE. I can't find any justification in the linked WebAuthn or CTAP specs for why this algorithm needs to exist at all. Section 5.3 says that it needs to be registered because some WebAuthn TPM attestations use it, but the very same section says that the algorithm MUST NOT be used by COSE implementations (is a WebAuthn implementation not a COSE implementation?). If the normative language in the spec is obeyed then the algorithm will never be used and so the registered identifier isn't needed. [JLS] For better or for worse, RS1 is already registered for JOSE, so that is the reason it is not registered here. Ouch, I hadn't seen this. The WebCrypto group really did a number on the registry. Thankfully most of them (including RS1) are only registered for JWK usage and marked as Prohibited. (What does it even mean for things like "A128CBC" to be registered as a JWK "alg" value?) [JLS] One can have a JWK which contains a symmetric key so in that case an βalgβ value of βA128CBCβ makes sense. Only use this key with this algorithm. My main point still stands that section 5.3 of the draft is self-contradictory as it says that the reason for registry is because some TPMs are using the algorithm but then also says that those implementations MUST NOT use the algorithm, negating the reason for registering it in the first place. [JLS] I agree and I have also pointed this out in a couple of reviews. -- Neil
_______________________________________________ jose mailing list [email protected] https://www.ietf.org/mailman/listinfo/jose
