[Insipid] Review of draft-ietf-insipid-logme-reqs-11

Dan Romascanu <dromasca@gmail.com> Thu, 12 January 2017 16:17 UTC

Return-Path: <dromasca@gmail.com>
X-Original-To: insipid@ietf.org
Delivered-To: insipid@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 0DCDD1294C9; Thu, 12 Jan 2017 08:17:17 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Dan Romascanu <dromasca@gmail.com>
To: <ops-dir@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148423783701.29293.15508238136822500989.idtracker@ietfa.amsl.com>
Date: Thu, 12 Jan 2017 08:17:17 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/insipid/PsgkRiWdTLWwW1TBljcgcixku5s>
Cc: dromasca@gmail.com, insipid@ietf.org, ietf@ietf.org, draft-ietf-insipid-logme-reqs.all@ietf.org
Subject: [Insipid] Review of draft-ietf-insipid-logme-reqs-11
X-BeenThere: insipid@ietf.org
X-Mailman-Version: 2.1.17
List-Id: SIP Session-ID discussion list <insipid.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/insipid>, <mailto:insipid-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/insipid/>
List-Post: <mailto:insipid@ietf.org>
List-Help: <mailto:insipid-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/insipid>, <mailto:insipid-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 12 Jan 2017 16:17:17 -0000

Reviewer: Dan Romascanu
Review result: Ready

I have reviewed this document as part of the Operational directorate's

ongoing effort to review all IETF documents being processed by the
IESG.  These 
comments were written with the intent of improving the operational
aspects of the 
IETF drafts. Comments that are not addressed in last call may be
included in AD reviews 
during the IESG review.  Document editors and WG chairs should treat
these comments 
just like any other last call comments. 

   This informational document describes requirements for adding an
indicator to the SIP
   protocol data unit (PDU, or a SIP message) that marks the PDU as a
   candidate for logging.  Such marking will typically be applied as
   part of network testing controlled by the network operator and not
   used in regular client signaling.  However, such marking can be
   carried end-to-end including the SIP terminals, even if a session
   originates and terminates in different networks.

It's a short, focused and well-written document. It is interesting for
the operators of networks that use SIP, as this indicator can be used
to trigger testing or debugging in the networks that they operate. It
seems that different implications on security and network behavior
were considered, although some of them are not explicitly mentioned -
such as the potential overload or DoS attacks in case of
mis-configuration or malicious configuration of a big number of
terminals in a SIP network leading to simultaneous activation of debug
modes. It may be good to explicitly mentions these, but otherwise this
document is READY from an OPS-DIR perspective.