[secdir] Secdir last call review of draft-ietf-sipcore-originating-cdiv-parameter-05
Kyle Rose <krose@krose.org> Mon, 22 October 2018 20:45 UTC
Return-Path: <krose@krose.org>
X-Original-To: secdir@ietf.org
Delivered-To: secdir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 53695130F1D; Mon, 22 Oct 2018 13:45:57 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Kyle Rose <krose@krose.org>
To: secdir@ietf.org
Cc: sipcore@ietf.org, ietf@ietf.org, draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.87.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <154024115730.13565.4561513275762767750@ietfa.amsl.com>
Date: Mon, 22 Oct 2018 13:45:57 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/g-d-XIh4M09sXO5MIaZhjysn3d8>
Subject: [secdir] Secdir last call review of draft-ietf-sipcore-originating-cdiv-parameter-05
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>, <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>, <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 22 Oct 2018 20:46:03 -0000
Reviewer: Kyle Rose Review result: Ready I have reviewed this document as part of the security directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written primarily for the benefit of the security area directors. Document editors and WG chairs should treat these comments just like any other last call comments. Background: SIP currently provides no standard way to indicate that an invite via diversion from the original terminating user (e.g., via busy response or an unconditional call forward) is the result of diversion rather than a new origination, making it impossible in the absence of proprietary signaling to distinguish between the two cases. This draft adds an orig-cdiv signal to the P-Served-User header to indicate an invite resulting from a diversion, allowing an application server to take different actions in the two cases. Security evaluation: The addition of a new actionable signal is properly noted as the main security consideration of this change, along with a reminder that the P-Served-User header must be implemented as a privileged channel within a single trust domain and not transited into a domain from an untrusted entity. Other comments: As a reviewer new to SIP, I found the document very easy to follow. The introduction contained basic background about the use case, a clear explanation of the problem, and a high-level overview of the proposed solution. The normative section of the document was very clearly illustrated by the later examples section, containing two typical use-cases with clear diagrams. This is a model internet draft.