[lamps] FW: WGLC: draft-ietf-lamps-pkix-shake-02

[Jim Schaad <[e-mail]>]

Forgot to add to the distribution list.

-----Original Message-----
From: Jim Schaad <[e-mail]> 
Sent: Friday, August 3, 2018 12:40 PM
To: '[e-mail]'
<[e-mail]>
Subject: WGLC: draft-ietf-lamps-pkix-shake-02

Not ready for progression.

* Run the NITS on this document and fix them.  Examples of problems are the
fact that MUST language section is missing, possible incorrect references,
and you have lines that are too long.

*  Introduction - I have a problem with the cardinality of items in the
second and third paragraphs here.  I do not ask that you fix the problems
that I have but you should be ready to address this is you get the same
questions from the RFC Editor or the IESG.  I would consider SHAKE to be a
family of extendable-output hash functions and thus has a cardinality of
one.  The two specific hash functions have a cardinality of greater than
one.  The question of cardinality comes in terms of the usage of 'A', 'is',
'are'.

* Introduction - paragraph 2 - I find the last sentence to be difficult to
read.  The usage of 'and' here seems to be incorrect and it may be difficult
to figure out which pair comes first - resistance or function.

* Introduction - paragraph 3 - I am unaware that ECDSA has a mask generating
function associated with it.  This sentence needs to be cleaned up

* Introduction - paragraph 3 - Consider putting in a reference to the
algorithm identifiers that are not changing.  Probably overkill but still
useful

* Identifiers - This section needs to nail down all parameters associated w/
the different SHAKE functions when used here.  Otherwise you end up with the
first assumption that I made which was d = 128 for SHAKE128 which would not
produce an acceptable result.

* Signatures - Para #3 - you refer to section 3 for OIDs, but they are not
there for public keys.

* IANA Considerations is incorrect and MUST be updated

* Why is there no reference to deterministic ECDSA signatures in the
document.

* The ASN.1 module is absent and needs to be instantiated.  Even doing so
with TBD is sufficient for now.

Jim