[Tsv-art] Tsvart last call review of draft-ietf-avtcore-cc-feedback-message-08

Michael Scharf via Datatracker <noreply@ietf.org> Wed, 09 September 2020 08:22 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: tsv-art@ietf.org
Delivered-To: tsv-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 4069F3A10E3; Wed, 9 Sep 2020 01:22:31 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Michael Scharf via Datatracker <noreply@ietf.org>
To: <tsv-art@ietf.org>
Cc: draft-ietf-avtcore-cc-feedback-message.all@ietf.org, last-call@ietf.org, avt@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.16.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <159963975122.30982.6287742761908271234@ietfa.amsl.com>
Reply-To: Michael Scharf <michael.scharf@hs-esslingen.de>
Date: Wed, 09 Sep 2020 01:22:31 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/QxQkHh-7D2IqZJRPBHUS3phazqs>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-avtcore-cc-feedback-message-08
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 09 Sep 2020 08:22:31 -0000

Reviewer: Michael Scharf
Review result: Ready with Issues

This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

Major issues
-------------

None

Minor issues
-------------

1/ Section 5:

   All RTP congestion control algorithms MUST specify how they
   respond to the loss of feedback packets.

This is a process-related requirement not relevant for interoperability of
implementations. In addition, the requirement is not very specific (What would
have to be specified?). I am not sure if such a requirement in capital letters
is really needed here. This should be handled consistently in all IETF
documents.

2/ Section 11:

The Security Considerations do not discuss off-path attacks, and it is not
clear why this case is missing. Can an off-path attacker trick the sender into
sending at either an excessively high or excessively low rate?

Nits
----

1/ Abstract:

The protocol extension enables fine-grained feedback on per-packet reception
quality. The rationale is provided in Section 1 and (more comprehensively) in
Section 8. Yet, I wonder if this objective could also be made a bit more
explicit in the abstract, e.g., along the lines of the "fine-grained feedback"
wording in the first paragraph of Section 8.

2/ Section 7:

Typo in "a=ecn-capaable-rtp:"