Re: [Last-Call] [lamps] Secdir last call review of draft-ietf-lamps-ocsp-nonce-03

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks for the Review comments, Sean. I will make these changes and submit a new version, I will wait till the deadline to see if there are any more comments before publishing new draft. 

Regards,
Mohit 

On Tue, Sep 1, 2020 at 7:10 AM Sean Turner via Datatracker <noreply@xxxxxxxx> wrote:
Reviewer: Sean Turner
Review result: Has Nits

The is short little document that put size constraints on the OCPS nonce
extension. In particular, it requires the nonce be between 1 and 32 octets
long. Since there's no bits on the wire changes, I think it's fine there's no
new ASN.1 module.

My technical nits include:

0. s2.1: Consider also throwing an error if the OCTET STRING is zero bytes in
length. You throw one if it's too long, but not if it's too short.

1. s2.1, 4th paragraph: Consider adding reference to RFC 4086 for randomness
requirements.

My editorial nits include:

0. Abstract: missing period at the end of paragraph.

1. s2.1, 3rd para: missing period at end of paragraph

2. s4: I think the indentation is off by a couple of characters.


_______________________________________________
Spasm mailing list
Spasm@xxxxxxxx
https://www.ietf.org/mailman/listinfo/spasm
-- 
last-call mailing list
last-call@xxxxxxxx
https://www.ietf.org/mailman/listinfo/last-call

[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux