Re: [Pce] Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05

Tero Kivinen <kivinen@iki.fi> Fri, 27 August 2021 13:11 UTC

Return-Path: <kivinen@iki.fi>
X-Original-To: expand-draft-ietf-lsr-pce-discovery-security-support.all@virtual.ietf.org
Delivered-To: pce@ietfa.amsl.com
Received: by ietfa.amsl.com (Postfix, from userid 65534) id 65AFF3A1571; Fri, 27 Aug 2021 06:11:31 -0700 (PDT)
X-Original-To: xfilter-draft-ietf-lsr-pce-discovery-security-support.all@ietfa.amsl.com
Delivered-To: xfilter-draft-ietf-lsr-pce-discovery-security-support.all@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C7FF33A156F for <xfilter-draft-ietf-lsr-pce-discovery-security-support.all@ietfa.amsl.com>; Fri, 27 Aug 2021 06:11:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.1
X-Spam-Level:
X-Spam-Status: No, score=-2.1 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, NICE_REPLY_A=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=iki.fi
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id j7ZuBVVQY6Ts for <xfilter-draft-ietf-lsr-pce-discovery-security-support.all@ietfa.amsl.com>; Fri, 27 Aug 2021 06:11:24 -0700 (PDT)
Received: from lahtoruutu.iki.fi (lahtoruutu.iki.fi [IPv6:2a0b:5c81:1c1::37]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E9D9A3A156B for <draft-ietf-lsr-pce-discovery-security-support.all@ietf.org>; Fri, 27 Aug 2021 06:11:23 -0700 (PDT)
Received: from fireball.acr.fi (fireball.kivinen.iki.fi [IPv6:2001:1bc8:100d::2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kivinen@iki.fi) by lahtoruutu.iki.fi (Postfix) with ESMTPSA id 064FC1B00120; Fri, 27 Aug 2021 16:11:17 +0300 (EEST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iki.fi; s=lahtoruutu; t=1630069877; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xAXltp6vIt9Q5wQDBnx5jJAhq83R7cwHDPMf5OKCUt0=; b=kZdDHUrDljPFN7M9JBZKq9UYA4qTDmFYHwooK+jhhc6z0K7AM/w7ooX60CLcYkXxnqt6My WKsC00ETwJjAObNVpiB+TdrVWPxCdeguTlgNfoq7UoKaDQKdBjd45qDdl1SWWlxQde57hw 16xP40KtC3Gp6Qyr2QRiaHWgsRQ/ue/wl1LXp4nuae9u8uQlc8O02/qTGD7yrcxMkvJJHB 52A1VFVLZXFTdOFs4S89tLstf26Xb9GblJYqckd6NhVXCiAu3rFqQJtNsicQrElcxzFvyK 6s6UH9efZrGSoFFo31FeXRfXbE7devomKVuy6dFFIW3ldOjIWEXb9aBuIOcR3g==
Received: by fireball.acr.fi (Postfix, from userid 15204) id 87FE725C120D; Fri, 27 Aug 2021 16:11:16 +0300 (EEST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Message-ID: <24872.58484.337833.575942@fireball.acr.fi>
Date: Fri, 27 Aug 2021 16:11:16 +0300
From: Tero Kivinen <kivinen@iki.fi>
To: Yaron Sheffer <yaronf.ietf@gmail.com>
Cc: "Acee Lindem (acee)" <acee@cisco.com>, Qin Wu <bill.wu@huawei.com>, tom petch <ietfc@btconnect.com>, "draft-ietf-lsr-pce-discovery-security-support.all@ietf.org" <draft-ietf-lsr-pce-discovery-security-support.all@ietf.org>
In-Reply-To: <96DF4CF9-CA27-4079-8080-757C60AFEC34@gmail.com>
References: <823a8f74b1824ff8827734446fe6f335@huawei.com> <CCF96710-139E-46C7-B615-1A7B74458862@gmail.com> <8BC3300E-2F4E-4965-8857-FC31BA7D1B40@cisco.com> <96DF4CF9-CA27-4079-8080-757C60AFEC34@gmail.com>
X-Mailer: VM 8.2.0b under 26.3 (x86_64--netbsd)
X-Edit-Time: 5 min
X-Total-Time: 4 min
ARC-Seal: i=1; s=lahtoruutu; d=iki.fi; t=1630069877; a=rsa-sha256; cv=none; b=sS4GZbjURmLLntIuj7f6XNCXyC8ox1FZgufOqqNlGoOMevviJdHjFrvG9Fyy9lOwmm1CiY G1kVXYCtXrs2KbtkVmOiWw2Lnv82JBS9rvGKbWDMxMi2wDmvEbR5gQ+Zal9H+5HIfi9sYb /w9YC1zeGM9lA6HjuwNp+0POCxgpW5zXJGxqc8UNI9AptS/sJ+pfd8Mzagkk0LdXrjp+bR IQQ2rJrJffDunACm7idEMTI3UVoVfYlxPkRMb6D0uynvRKfX0VJQrMk3brN1QEWQ0y8SI+ Z2yYdXuAtxuvcMKjrAu2OVmFy7vZ3d9LHgzIEIwOLhq27AkqAgcQ4Rq+1K1XLQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=iki.fi; s=lahtoruutu; t=1630069877; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xAXltp6vIt9Q5wQDBnx5jJAhq83R7cwHDPMf5OKCUt0=; b=X+ultc5YvoPBnHoEeb1bWrU8mxNIGqj8Y3SlLom3A1Wyb51gBVPzRZN+I3VYvqxyagq+Aw JtG9jAk7QTpxtlDPHTN5M73Q7ysqOaXm6GbgU7nPDdFSn0vuRLZurPW9HXHcTmtp/omW5J 5dLNK6GaX8kS56cYKibB6Sk4+ZCHZbbBGhmnha6ppxH9ROaVQqE5TaQn2XjBioABmWNT7D VNmK4ZoiIIUg4l+6S8NSviBrubN4UjKDakG4pF4pzPyebrwswfr69YKEEI5IFwPQA/eMc/ MAQKll5on8WjpZ6GEmgcSWpJyAFUCIGTJZBffFCO7NQh/wMRbPINgPBV5UEh+g==
ARC-Authentication-Results: i=1; ORIGINATING; auth=pass smtp.auth=kivinen@iki.fi smtp.mailfrom=kivinen@iki.fi
Resent-From: alias-bounces@ietf.org
Resent-To: pce@ietf.org, dhruv.ietf@gmail.com, acee@cisco.com, martin.vigoureux@nokia.com, jgs@juniper.net, lsr@ietf.org, bill.wu@huawei.com, yingzhen.ietf@gmail.com, daniel@olddog.co.uk, maqiufang1@huawei.com, chopps@chopps.org, aretana.ietf@gmail.com, diego.r.lopez@telefonica.com
Resent-Message-Id: <20210827131131.65AFF3A1571@ietfa.amsl.com>
Resent-Date: Fri, 27 Aug 2021 06:11:31 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/Gi33bFcdP3D-iCHLu1_wqjysEgQ>
Subject: Re: [Pce] Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 27 Aug 2021 13:11:32 -0000

Yaron Sheffer writes:
> Hi Acee,
> 
> I honestly don't know how to do it, and if I even can unless you
> send a new review request. 

We do not really update the reviews, but we do assign draft for two
reviews, i.e., one during the last call and one before the telechat (I
will not reassign it for telechat if there is no need to do rereview,
i.e., original review was ready and/or nothing major changed in the
draft since the review). 

> Copying Tero who's an expert on this.
> 
> To clarify: my review of -05 still stands, but it has been addressed
> by -07.

As area directors will see the review email thread and if the there is
comment there that review comments have been addressed in later drafts
that is enough, so no need to update the review.

Note, that even when the reviewer thinks his comments have been
addressed, the area director might have different view on that matter,
i.e., they might still comment on the issues found during the
telechat. 

> 
> Thanks,
> 	Yaron
> 
> 
> On 8/19/21, 20:55, "Acee Lindem (acee)" <acee@cisco.com> wrote:
> 
>     Hi Yaron,
> 
>     Thanks for the review. Can you update the status of the SECDIR review? 
> 
>     https://datatracker.ietf.org/doc/review-ietf-lsr-pce-discovery-security-support-05-secdir-lc-sheffer-2021-08-05/
> 
>     Thanks,
>     Acee
> 
>     On 8/17/21, 3:15 AM, "Yaron Sheffer" <yaronf.ietf@gmail.com> wrote:
> 
>         Looks good to me. Thank you!
> 
>         	Yaron
> 
>         On 8/17/21, 03:17, "Qin Wu" <bill.wu@huawei.com> wrote:
> 
>             Sorry for late followup, here is the update, the diff is
>             https://www.ietf.org/rfcdiff?url2=draft-ietf-lsr-pce-discovery-security-support-06
>             Yaron, let authors know if your comments are addressed in v-06.
>             Thanks!
> 
>             -Qin
>             -----邮件原件-----
>             发件人: Acee Lindem (acee) [mailto:acee@cisco.com] 
>             发送时间: 2021年8月17日 4:33
>             收件人: Qin Wu <bill.wu@huawei.com>; tom petch <ietfc@btconnect.com>; Yaron Sheffer <yaronf.ietf@gmail.com>; secdir@ietf.org
>             抄送: draft-ietf-lsr-pce-discovery-security-support.all@ietf.org
>             主题: Re: Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
> 
>             Hi Qin, 
> 
>             Can you publish a revision so that Yaron assure it satisfies his comments? 
> 
>             Thanks,
>             Acee
> 
>             On 8/12/21, 9:21 PM, "Qin Wu" <bill.wu@huawei.com> wrote:
> 
>                 Thanks Acee and Tom for good suggestion, we will take them into account.
> 
>                 -Qin
>                 -----邮件原件-----
>                 发件人: Acee Lindem (acee) [mailto:acee@cisco.com] 
>                 发送时间: 2021年8月12日 1:18
>                 收件人: tom petch <ietfc@btconnect.com>; Yaron Sheffer <yaronf.ietf@gmail.com>; Qin Wu <bill.wu@huawei.com>; secdir@ietf.org
>                 抄送: draft-ietf-lsr-pce-discovery-security-support.all@ietf.org
>                 主题: Re: Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
> 
>                 I'd also recommend changing, "key names" to "key-ids or key-chain names" since this is what is actually being advertised.
>                 Thanks,
>                 Acee
> 
>                 On 8/10/21, 11:53 AM, "tom petch" <ietfc@btconnect.com> wrote:
> 
>                     From: Lsr <lsr-bounces@ietf.org> on behalf of Yaron Sheffer <yaronf.ietf@gmail.com>
>                     Sent: 10 August 2021 14:57
> 
>                     So let me suggest:
> 
>                     <tp>
>                     An offlist suggestion for you to consider
> 
>                     OLD
>                         Thus before advertisement of the PCE security parameters, it MUST be insured that the IGP protects the authentication and integrity of the PCED TLV using the mechanisms defined in
>                         [RFC5310] and [RFC5709], if the mechanism described in this document is used.
> 
>                         Moreover, as stated in [RFC5088] and [RFC5089], the IGP do not provide any encryption mechanisms to protect the secrecy of the PCED TLV, and the operator must ensure that no private data is carried in the TLV, for example that key names do not reveal sensitive information about the network.
> 
>                     NEW
> 
>                      Thus before advertising the PCE security parameters, using the mechanism described in this document, the IGP MUST be known to provide authentication and integrity for the PCED TLV using the mechanisms defined in  [RFC5304],  [RFC5310] or [RFC5709],
> 
>                         Moreover, as stated in [RFC5088] and [RFC5089], if the IGP does not provide any encryption mechanisms to protect the secrecy of the PCED TLV, then the operator must ensure that no private data is carried in the TLV, e.g. that key names do not reveal sensitive information about the network.
> 
>                     Tom Petch
>                     </tp>
> 
>                     Thanks,
>                             Yaron
> 
>                     On 8/10/21, 15:01, "Qin Wu" <bill.wu@huawei.com> wrote:
> 
>                         Yaron:
>                         Thank for clarification. I agree to keep the last sentence in the second paragraph of section 7 as is.
>                         But I prefer to add the addition references in the previous sentence as follows:
>                         "
>                         Thus before advertisement of the PCE security parameters, it MUST be insured that the IGP is
>                         protected for authentication and integrity of the PCED TLV,, with the mechanisms defined in
>                         [RFC5310] and [RFC5709] if the mechanism described in this document is used.
> 
>                         As stated in [RFC5088] and [RFC5089], the IGP do not provide encryption mechanism to protect
>                         the privacy of the PCED TLV, if this information can make the PCEP session less secure then the operator should take that into consideration.
>                         "
>                         If you better wording, please let me know.
> 
>                         -Qin
>                         -----邮件原件-----
>                         发件人: Yaron Sheffer [mailto:yaronf.ietf@gmail.com]
>                         发送时间: 2021年8月10日 19:26
>                         收件人: Qin Wu <bill.wu@huawei.com>; secdir@ietf.org
>                         抄送: draft-ietf-lsr-pce-discovery-security-support.all@ietf.org; last-call@ietf.org; lsr@ietf.org
>                         主题: Re: Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
> 
>                         Hi Qin,
> 
>                         Sorry, but I find your latest proposed text very confusing, because we should be focusing on integrity protection and not privacy (=secrecy) of the TLV. So I would prefer to keep the text as-is, with the addition of a reference to the IS-IS and OSPF security mechanisms that were discussed on this thread.
> 
>                         Thanks,
>                             Yaron
> 
>                         On 8/10/21, 05:00, "Qin Wu" <bill.wu@huawei.com> wrote:
> 
>                             Hi, Yaron
>                             -----邮件原件-----
>                             >发件人: Yaron Sheffer [mailto:yaronf.ietf@gmail.com]
>                             >发送时间: 2021年8月9日 21:44
>                             >收件人: Qin Wu <bill.wu@huawei.com>; secdir@ietf.org
>                             >抄送: draft-ietf-lsr-pce-discovery-security-support.all@ietf.org; last-call@ietf.org; lsr@ietf.org
>                             >主题: Re: Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
> 
>                             >Hi Qin,
> 
>                             >Thank you for your response.
> 
>                             >* RFC 3567 (for IS-IS) is obsoleted by RFC 5304. Unfortunately RFC 5304 still uses HMAC-MD5, which would be considered insecure nowadays.
>                             >* RFC 2154 is very old and Experimental (and only supports RSA-MD5 signatures). I'm not an OSPF expert by any means, but I'm willing to bet that there are no production implementations of this RFC. (I'm willing to be proven wrong).
>                             >Is there another RFC that define a protection mechanism for OSPF?
> 
>                             >All in all, there appear to be no good options for the IGP.
> 
>                             [Qin Wu]Yes, we do have alternatives, see Les's response in the separate email
>                             "
>                             On 8/9/21, 23:36,"Les Ginsberg (ginsberg)" <ginsberg@cisco.com> wrote:
>                             For IS-IS security please also see RFC 5310.
>                             For OSPF security please see RFC 5709.
>                             "
>                             >To your last point, when I mentioned decoupling the mechanisms, I was suggesting to use the extension you define even if the IGP *cannot* be secured. If you think this is reasonable, please add such text to the Security Considerations.
> 
>                             [Qin Wu] Okay, how about the following change
>                             OLD TEXT:
>                             "
>                             As stated in [RFC5088]
>                             and [RFC5089], the IGP do not provide encryption mechanism to protect
>                             the privacy of the PCED TLV, if this information can make the PCEP
>                             session less secure then the operator should take that into consideration .
>                             "
>                             NEW TEXT:
>                             "
>                             As stated in [RFC5088]
>                             and [RFC5089], the IGP do not provide encryption mechanism to protect
>                             the privacy of the PCED TLV, if this information can make the PCEP
>                             session less secure then the operator should take that into consideration
>                             when getting the mechanism described in this document deployed.
>                             "
>                              >Thanks,
>                              >      Yaron
> 
>                             >On 8/9/21, 16:09, "Qin Wu" <bill.wu@huawei.com> wrote:
> 
>                               >   Thanks Yaron for valuable comments, please see my reply inline below.
>                                 -----邮件原件-----
>                                 >发件人: Yaron Sheffer via Datatracker [mailto:noreply@ietf.org]
>                                 >发送时间: 2021年8月6日 3:25
>                                 >收件人: secdir@ietf.org
>                                 >抄送: draft-ietf-lsr-pce-discovery-security-support.all@ietf.org; last-call@ietf.org; lsr@ietf.org
>                                 >主题: Secdir last call review of draft-ietf-lsr-pce-discovery-security-support-05
> 
>                                 >Reviewer: Yaron Sheffer
>                                 >Review result: Not Ready
> 
>                                 >This document defines a mechanism (a TLV) to advertise the PCE Protocol security required (use of TCP-AO and its key ID, or alternatively use of TLS) within the routing protocol being used.
> 
>                                 >* Sec. 3.1: I don't understand why "SHOULD advertise" and not MUST. Especially given the strict client behavior defined later.
>                                 [Qin]: I believe "SHOULD advertise" is consistent with client behavior defined later, i.e., we apply SHOULD NOT language to the client behavior.
>                                 I am not sure we should change it into strong language with MUST. Since if IGP advertisement doesn't include TCP-AO
>                                  support flag bit or TLS support flag bit, NMS may fall back to configure both PCC and PCE server to support TCP-AO or TLS. That's one of reason I think why we choose to use SHOULD language.
> 
>                                 >* Sec. 3.1: should we also say something about the case where both methods are advertised, and whether we recommend for the client to use one of them over the other?
> 
>                                 [Qin]: It is up to local policy, which has bee clarified in the end of section 3.1. Hope this clarify.
> 
>                                 >* Sec. 4: typo (appears twice) - "to be carried in the PCED TLV of the for use".
> 
>                                 [Qin]:Thanks, have fixed them in the local copy.
> 
>                                 >* Sec. 7: this phrase appears to be essential to security of this mechanism: "it MUST be insured that the IGP is protected for authentication and integrity of the PCED TLV". I would expect more guidance: how can this property be ensured in the relevant IGPs?
>                                 [Qin]:I think mechanism defined in [RFC3567] and [RFC2154] can be used to ensure authenticity and integrity of OSPF LSAs or ISIS LSPs and their TLVs. Here is the proposed changes:
>                                 OLD TEXT:
>                                 "
>                                    Thus before advertisement of
>                                    the PCE security parameters, it MUST be insured that the IGP is
>                                    protected for authentication and integrity of the PCED TLV if the
>                                    mechanism described in this document is used.
>                                 "
>                                 NEW TEXT:
>                                 "
>                                    Thus before advertisement of
>                                    the PCE security parameters, it MUST be insured that the IGP is
>                                    protected for authentication and integrity of the PCED TLV with mechanisms defined in [RFC3567][RFC2154] if the
>                                    mechanism described in this document is used.
>                                 "
>                                 >* Also, a possibly unintended consequence of this requirement is that if the IGP cannot be protected in a particular deployment/product, this mechanism would not be used. Please consider if this is likely to happen and whether we want to forego PCEP transport >security in such cases. My gut feel (not based on experience in such networks) is that the threat models are different enough that we should decouple the security of IGP from that of PCEP.
> 
>                                 [Qin] I agree IGP security should be separated from PCEP security. IGP extension defined in this document is used by the PCC to select PCE server with appropriate security mechanism. On the other hand, Operator can either use IGP advertisement for PCEP security capability or rely on local policy to select PCE. If operator feels IGP advertisement is not secure, he can fall back to local policy or rely on manual configuration. Hope this clarifies.
> 
> 
> 
> 
> 
> 
> 
>                     _______________________________________________
>                     Lsr mailing list
>                     Lsr@ietf.org
>                     https://www.ietf.org/mailman/listinfo/lsr
> 
> 
> 
> 
> 
> 

-- 
kivinen@iki.fi