Re: [Sidrops] Rtgdir telechat review of draft-ietf-sidrops-bgpsec-algs-rfc8208-bis-04

"Carlos Pignataro (cpignata)" <cpignata@cisco.com> Fri, 12 April 2019 04:41 UTC

Return-Path: <cpignata@cisco.com>
X-Original-To: sidrops@ietfa.amsl.com
Delivered-To: sidrops@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7DE551201EF; Thu, 11 Apr 2019 21:41:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -12.511
X-Spam-Level:
X-Spam-Status: No, score=-12.511 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, HTTPS_HTTP_MISMATCH=1.989, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
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 j2G9oaN04J1K; Thu, 11 Apr 2019 21:41:27 -0700 (PDT)
Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id AEF101201D1; Thu, 11 Apr 2019 21:41:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=19522; q=dns/txt; s=iport; t=1555044086; x=1556253686; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=xnsziRIMGQxpnzaI5Q6b9tnyvMERo5gQmkHiTFgXY68=; b=L31itZPfQkkLyWwlbgFA1YqE6mx0eoSzYFRYMKM2lxtmsI/ufnY36IGh 2gCVBpBLRMmFiZw2m1VbaJR3G3sYF6YUZqRFySWCoe133Gjk7gBYSlV7S yeb6493OPyMqUrgLiibq0Cr1w0kkuOI0t2xqtDX5Sh2W5JhIHdyQUmNa3 I=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0AFAADuFbBc/5FdJa1lGgEBAQEBAgE?= =?us-ascii?q?BAQEHAgEBAQGBUgQBAQEBCwGBDlgqaIEDKAqEBJU2kk+FeYF7DgEBIoEQXYJ?= =?us-ascii?q?eAheFXCM1CA0BAQMBAQoBAgECbRwMhUoBAQEEI08HDAQCAQgRBAEBKAMCAgI?= =?us-ascii?q?wFAkIAgQOBYMdBAEBJgF2TAMcD6wGgS+EMQGDUQOCK4EwAYRehmgXgUA/gRE?= =?us-ascii?q?nH4JMPoJhAgIYgUYYgnMxgiYDinACBYIohDGHYIxtCQKCBYU2UYhHg0MTB4I?= =?us-ascii?q?GXYU9jE6MeYEBhAyKcYJ0AhEVgTAhAjQNgUlwFTsqAYJBCQorhTuFFIU/QTE?= =?us-ascii?q?BAQEBjh4rgQQybgEB?=
X-IronPort-AV: E=Sophos;i="5.60,340,1549929600"; d="scan'208,217";a="260675811"
Received: from rcdn-core-9.cisco.com ([173.37.93.145]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 12 Apr 2019 04:41:25 +0000
Received: from XCH-RTP-019.cisco.com (xch-rtp-019.cisco.com [64.101.220.159]) by rcdn-core-9.cisco.com (8.15.2/8.15.2) with ESMTPS id x3C4fO31012891 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 12 Apr 2019 04:41:25 GMT
Received: from xch-rtp-020.cisco.com (64.101.220.160) by XCH-RTP-019.cisco.com (64.101.220.159) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 12 Apr 2019 00:41:24 -0400
Received: from xch-rtp-020.cisco.com ([64.101.220.160]) by XCH-RTP-020.cisco.com ([64.101.220.160]) with mapi id 15.00.1473.003; Fri, 12 Apr 2019 00:41:24 -0400
From: "Carlos Pignataro (cpignata)" <cpignata@cisco.com>
To: "Borchert, Oliver (Fed)" <oliver.borchert@nist.gov>
CC: Routing Directorate <rtg-dir@ietf.org>, "sidrops@ietf.org" <sidrops@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "draft-ietf-sidrops-bgpsec-algs-rfc8208-bis.all@ietf.org" <draft-ietf-sidrops-bgpsec-algs-rfc8208-bis.all@ietf.org>
Thread-Topic: Rtgdir telechat review of draft-ietf-sidrops-bgpsec-algs-rfc8208-bis-04
Thread-Index: AQHU5Aq8UAmlfJbAlUyDGzfunZHNJKY3jhHwgADA3YA=
Date: Fri, 12 Apr 2019 04:41:23 +0000
Message-ID: <E2780165-6EBA-4D5F-8CB5-18E47ACD6B4C@cisco.com>
References: <155362877270.7408.1659232059641306508@ietfa.amsl.com> <SN6PR09MB31674C565BB1F36200F1AFA6982F0@SN6PR09MB3167.namprd09.prod.outlook.com>
In-Reply-To: <SN6PR09MB31674C565BB1F36200F1AFA6982F0@SN6PR09MB3167.namprd09.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-mailer: Apple Mail (2.3445.104.8)
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.118.116.132]
Content-Type: multipart/alternative; boundary="_000_E27801656EBA4D5F8CB518E47ACD6B4Cciscocom_"
MIME-Version: 1.0
X-Outbound-SMTP-Client: 64.101.220.159, xch-rtp-019.cisco.com
X-Outbound-Node: rcdn-core-9.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/sidrops/7qHl4TIxXrw_e4ed2fAHcsYJ3yQ>
X-Mailman-Approved-At: Sun, 14 Apr 2019 09:41:45 -0700
Subject: Re: [Sidrops] Rtgdir telechat review of draft-ietf-sidrops-bgpsec-algs-rfc8208-bis-04
X-BeenThere: sidrops@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: A list for the SIDR Operations WG <sidrops.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sidrops>, <mailto:sidrops-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sidrops/>
List-Post: <mailto:sidrops@ietf.org>
List-Help: <mailto:sidrops-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sidrops>, <mailto:sidrops-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 12 Apr 2019 04:41:29 -0000

Thank you for having considered my comments, Oliver, and acting on them.

Thanks,

— Carlos Pignataro

On Apr 11, 2019, at 5:16 PM, Borchert, Oliver (Fed) <oliver.borchert@nist.gov<mailto:oliver.borchert@nist.gov>> wrote:

Carlos,

I fixed the NextHop IPv6 address to use a private use IP. It was an easy fix which does not affect the signatures.
I followed your advise to split the Special-Use ID into a range for Experimental-ID and Documentation-ID.

The topic of obsolete vs update is still open and I wait for IESG guidance.

If wanted/requested, I can upload a version 05 of the document with the modifications I outlined in the previous emails already.

Oliver


-----Original Message-----
From: Carlos Pignataro via Datatracker <noreply@ietf.org<mailto:noreply@ietf.org>>
Sent: Tuesday, March 26, 2019 3:33 PM
To: rtg-dir@ietf.org<mailto:rtg-dir@ietf.org>
Cc: sidrops@ietf.org<mailto:sidrops@ietf.org>; ietf@ietf.org<mailto:ietf@ietf.org>; draft-ietf-sidrops-bgpsec-algs-rfc8208-bis.all@ietf.org<mailto:draft-ietf-sidrops-bgpsec-algs-rfc8208-bis.all@ietf.org>
Subject: Rtgdir telechat review of draft-ietf-sidrops-bgpsec-algs-rfc8208-bis-04
Importance: High

Reviewer: Carlos Pignataro
Review result: Has Issues

Hello,

I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs.
For more information about the Routing Directorate, please see
​https://gcc01.safelinks.protection.outlook.com/?url=http%3A%2F%2Ftrac.tools.ietf.org%2Farea%2Frtg%2Ftrac%2Fwiki%2FRtgDir&amp;data=02%7C01%7Coliver.borchert%40nist.gov%7C4350e5e2db9141602a4008d6b221dd15%7C2ab5d82fd8fa4797a93e054655c61dec%7C1%7C0%7C636892255859405648&amp;sdata=0G9DxFL9CJzsYDQXEo%2Feo2IoThB%2Fmnay%2BjoIf2zQslw%3D&amp;reserved=0

Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft.

Document: draft-ietf-sidrops-bgpsec-algs-rfc8208-bis-04
Reviewer: Carlos Pignataro
Intended Status: Proposed Standard

Summary:
This document specifies the algorithms and parameters for BGPsec (Border Gateway Protocol Security).

Comments:
This is a clear, comprehensive, and well written document. It states it updates (if approved) RFC 8208, and I particularly appreciate Section 1.2, "Changes from RFC 8208", in explicitly showing how. However, it is unclear to me if the right relationship is to "update" or to "obsolete" RFC 8208. Should this document be approved and published, would RFC 8208 still be active and relevant, only updated, or re-written?

Minor Issues:

1.  Introduction

  This document updates [RFC7935] to add support for a) a different
  algorithm for BGPsec certificate requests, which are issued only by
  BGPsec speakers; b) a different Subject Public Key Info format for

CMP: Does this document update RFC7935 or RFC8208 on these issues? Meaning, if it really updates RFC7935, then it would obsolete RFC 8208. If it does not obsolete RFC 8208, then it would update RFC 8208 and RFC 7935, perhaps? CMP: I believe the right metadata would be: Updates: 7935 Obsoletes: 8208

CMP: Also, an editorial: this is a very thick paragraph to parse containing an enumerated list embedded in it. Should clarity be improved if turned into an actual list? (a), (b), etc.

  Appendix A contains example BGPsec UPDATE messages as well as the
  private keys used to generate the messages and the certificates
  necessary to validate the signatures.

CMP: Maybe overkill, but might be useful to explicitly say that the Appendix is non-normative. Just a thought for your consideration.

2.1. Algorithm ID Types

  o  Special-Use Algorithm ID

     Special-Use algorithm IDs span from 0xFA (250) to 0xFE (254).  To
     allow documentation and experimentation to accurately describe

CMP: I was wondering if it is appropriate to use a common block for both documentation (paper) and experimentation (wire in labs). CMP: In this, I note that RFC 4727 says:

"  It is not
  appropriate to use addresses in the documentation prefix [RFC3849]
  for experimentation."

CMP: So, while I have no strong position (I think), it might be useful to consider separating these two semantics with different allocations.

8.  References

CMP: Lastly, I am sure ADs are checking downrefs and the such.

Also Nits:

Found possible IPv6 address '2001:0010:0000:0000:0000:0000:c633:6464' in position 783 in the paragraph; this doesn't match RFC 3849's suggested
2001:DB8::/32 address range or RFC 4193's Unique Local Address range FC00::/7.

CMP: I hope these are useful.

Thank you,

Carlos Pignataro.