Skip to main content

Last Call Review of draft-ietf-avtcore-rtp-evc-05
review-ietf-avtcore-rtp-evc-05-secdir-lc-turner-2023-10-11-00

Request Review of draft-ietf-avtcore-rtp-evc
Requested revision No specific revision (document currently at 07)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2023-10-04
Requested 2023-09-20
Authors Shuai Zhao , Stephan Wenger , Youngkwon Lim
I-D last updated 2023-10-11
Completed reviews Artart Last Call review of -05 by Marc Blanchet (diff)
Secdir Last Call review of -05 by Sean Turner (diff)
Assignment Reviewer Sean Turner
State Completed
Request Last Call review on draft-ietf-avtcore-rtp-evc by Security Area Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/secdir/esVnyFLNZ2K5AxCgXVEmzYb4Lzg
Reviewed revision 05 (document currently at 07)
Result Has issues
Completed 2023-10-11
review-ietf-avtcore-rtp-evc-05-secdir-lc-turner-2023-10-11-00
tl;dr: Just one issue that I'll get to after rambling for a bit.

This is your typical I-D for an RTP Payload Format for foo. It contains the
usual disclaimers in the Security Considerations section that are found in RTP
Payload Format RFCs:

* It's just about the payload format
* Read RTP & Options for Securing RTP
* There's no MTI security solution (see RFC 7202)
* Apps SHOULD provide a strong security mechanism

This I-D, like RFC 7798, also includes considerations for:
* DoS concerns during compression
* SEI
* End-to-End Security

Issue: If this I-D is like RFC 7798, why does RFC 7798 say this:

 Therefore, the usage of data origin authentication and data integrity
 protection of at least the RTP packet is RECOMMENDED, for example,
 with SRTP [RFC3711].

And this I-D says this:

 Therefore, the usage of data origin authentication and data integrity
 protection of at least the RTP packet is RECOMMENDED but NOT REQUIRED
 based on the thoughts of [RFC7202].

It seems like this I-D says it's similar to HEVC, but then adds this little bit
extra.  Also, "NOT REQUIRED" isn't BCP 14 language so it's probably got to be
changed by either rewording or making it lower case.

Editorial:
* s9 (missing period): s/avoid those/avoid those.