[Last-Call] Artart last call review of draft-ietf-netmod-rfc6991-bis-16

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

 



Reviewer: Bron Gondwana
Review result: Not Ready

I'm the designated ARTART reviewer for this document.  It's generally well
written and clear; I didn't see any issues with the document itself, however
there are some obsolete references and changes to align with work done
elsewhere in the IETF which I believe would improve the overall
cross-compatibility of IETF specifications significantly, hence my marking it
as "NOT READY".

*Date-Time:*

While the date-time format duplicates the description found in RFC 6021 and
later RFC 6991, the construct -00:00 has been identified as being incompatible
with the latest ISO8601 by the work in the SEDATE working group.  I would refer
you to section 2 of RFC 9557 for the full description and update of RFC 3339
which was done there.  I suggest that this document should be updated to align
with (and reference) RFC 9557 and deprecate the usage of -00:00; instead using
"Z" to mean "local time reference point is unknown" as is common practice. 
This will improve future interoperability with ISO8601.

Likewise the same issue occurs with the new "date" format and "time" format.

*Email Address:*

The email-address construct in this document is limited to 7-bit.  RFC 6531 and
RFC 6532 have extended Email Address to allow UTF-8 characters.  There's a good
analysis of the changes at:
https://gist.github.com/baker-ling/3b4b014ee809aa9732f9873fe060c098

Since this is a new datatype being added, it should support all legal email
addresses as defined in current IETF RFCs; so be extended for 8 bit
local-parts. Similarly, the domain part of the address should explicitly
mention A-labels for Internationalized domain names, as the "domain-name"
construct does.



-- 
last-call mailing list -- last-call@xxxxxxxx
To unsubscribe send an email to last-call-leave@xxxxxxxx




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

  Powered by Linux