[Insipid] Suresh Krishnan's Discuss on draft-ietf-insipid-session-id-26: (with DISCUSS and COMMENT)

["Suresh Krishnan" <[e-mail]>]

Suresh Krishnan has entered the following ballot position for
draft-ietf-insipid-session-id-26: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-insipid-session-id/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

* Section 5

I do have a concern about backward compatibility regarding the sess-uuid.
Looks like this document allows the sess-uuid to contain either uppercase
or lowercase hex digits ("sess-uuid           = 32(DIGIT / %x41-46 /
%x61-66)") while the legacy version in RFC7329 does not allow uppercase
hex digits. Looks like a compliant implementation of the spec using upper
case hex digits will fail to interoperate with a legacy implementation. I
do not have a particular preference, but either this rule needs to be
tightened or there needs to be some text added to Section 11 to say this
will cause an interoperability issue.


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

* Section 4.1

The namespace ID in section 4.1 is specified as the initialization of a C
structure as described in Appendix C of RFC4122. It is missing a
semicolon at the end to make it legal.

OLD:
       uuid_t NameSpace_SessionID = {
           /* a58587da-c93d-11e2-ae90-f4ea67801e29 */
           0xa58587da,
           0xc93d,
           0x11e2,
           0xae, 0x90, 0xf4, 0xea, 0x67, 0x80, 0x1e, 0x29
       }

NEW:
       uuid_t NameSpace_SessionID = {
           /* a58587da-c93d-11e2-ae90-f4ea67801e29 */
           0xa58587da,
           0xc93d,
           0x11e2,
           0xae, 0x90, 0xf4, 0xea, 0x67, 0x80, 0x1e, 0x29
       };

* Agree with Alissa's DISCUSS regarding persistence