[Last-Call] Secdir last call review of draft-ietf-bier-evpn-13

Mohit Sethi via Datatracker <noreply@ietf.org> Mon, 01 January 2024 19:38 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: last-call@ietf.org
Delivered-To: last-call@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 09A7EC14F71B; Mon, 1 Jan 2024 11:38:38 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Mohit Sethi via Datatracker <noreply@ietf.org>
To: secdir@ietf.org
Cc: bier@ietf.org, draft-ietf-bier-evpn.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <170413791802.53656.1714323508767750456@ietfa.amsl.com>
Reply-To: Mohit Sethi <mohit@iki.fi>
Date: Mon, 01 Jan 2024 11:38:38 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/last-call/-znVwBzGq4QP4zDr3XUTiowCfx0>
Subject: [Last-Call] Secdir last call review of draft-ietf-bier-evpn-13
X-BeenThere: last-call@ietf.org
X-Mailman-Version: 2.1.39
List-Id: IETF Last Calls <last-call.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/last-call>, <mailto:last-call-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/last-call/>
List-Post: <mailto:last-call@ietf.org>
List-Help: <mailto:last-call-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/last-call>, <mailto:last-call-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 01 Jan 2024 19:38:38 -0000

Reviewer: Mohit Sethi
Review result: Has Nits

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.

This document defines procedures for forwarding broadcast, unknown unicast, and
multicast (BUM) traffic of Ethernet VPNs (EVPN) using Bit Index Explicit
Replication (BIER).

I am not at all familiar with this area and found the document somewhat
difficult to parse and comprehend. However, that is not necessarily a problem
since I am not the target audience in any case.

Some nits from my limited understanding:

The introduction mentions P-tunnels but it is explained a little later in
section 1.1.

I am not sure how should I interpret "As such, this document is also very much
aligned with [RFC8556]." What is RFC8556 and in what sense is the current draft
aligned?

Perhaps a reason or justification for this reuse could be helpful: "The same
codepoint 0x0B that IANA has assigned for BIER for MVPN [RFC8556] is used for
EVPN as well."

Several acronyms such as NLRI,mLDP P2MP NVGRE, GENEVE, BD, VNI/VSID are used
without expanding them on first use and without any definition in the
terminology section?

The document is missing the boiler plate text and reference to BCP 14 [RFC2119]
[RFC8174].

Considering BCP 14, how should one interpret the phrase "they do NOT apply to
EVPN"? Perhaps SHALL NOT or SHOULD NOT?

The security considerations section simply provides references to [RFC7432] and
[RFC8556] for security implications. I guess that is okay, but I noticed that
[RFC8556] further references [RFC8279] and [RFC8296] for security
considerations. Perhaps that is acceptable. One could consider stating that the
security of this solution is based on the full trust of the complete end-to-end
BIER network. There is no cryptography to ensure that a packet is not
manipulated enroute and properties such as integrity confidentiality of the
traffic is ensured at higher layers?