-08 after another mohamed review

This commit is contained in:
Randy Bush 2022-06-01 10:57:35 -07:00
parent 89b6284530
commit 6d734dcd3b

View file

@ -10,7 +10,7 @@
<?rfc compact="yes"?> <?rfc compact="yes"?>
<?rfc subcompact="no"?> <?rfc subcompact="no"?>
<rfc category="std" docName="draft-ietf-sidrops-8210bis-07" submissionType="IETF" updates="8210" ipr="trust200902" consensus="yes"> <rfc category="std" docName="draft-ietf-sidrops-8210bis-08" submissionType="IETF" updates="8210" ipr="trust200902" consensus="yes">
<front> <front>
@ -415,8 +415,8 @@
whether this PDU announces a new Router Key or deletes one whether this PDU announces a new Router Key or deletes one
previously announced Router Key PDU with the exact same AS previously announced Router Key PDU with the exact same AS
Number, subjectKeyIdentifier, and Number, subjectKeyIdentifier, and
subjectPublicKeyInfo.</t> subjectPublicKeyInfo.
</t>
<t> <t>
The remaining bits in the Flags field are reserved for The remaining bits in the Flags field are reserved for
future use. future use.
@ -449,8 +449,8 @@
subjectPublicKeyInfo SEQUENCE. subjectPublicKeyInfo SEQUENCE.
</t> </t>
<t hangText="Refresh Interval:"> <t hangText="Refresh Interval:">
Interval between normal cache polls. Interval between normal cache polls. See <xref
See <xref target="timing"/>. target="timing"/>.
</t> </t>
<t hangText="Retry Interval:"> <t hangText="Retry Interval:">
Interval between cache poll retries after a failed cache poll. Interval between cache poll retries after a failed cache poll.
@ -461,6 +461,26 @@
valid in the absence of a successful subsequent cache poll. valid in the absence of a successful subsequent cache poll.
See <xref target="timing"/>. See <xref target="timing"/>.
</t> </t>
<t hangText="AFI Flags:">
A field of the ASPA PDU where the low order bit denotes
whether the AS relationships are for IPv4 (0) or IPv6 (1)
AFI.
</t>
<t hangText="Provider AS Count:">
The number of Provider Autonomous System Numbers in the
PDU.
</t>
<t hangText="Customer Autonomous System Number:">
The AS number of the Autonomous System that authorizes the
upstream providers listed in the Provider Autonomous
System list to propagate prefixes of the specified address
family other ASes.
</t>
<t hangText="Provider Autonomous System Numbers:">
The set of AS numbers authorized to propagate prefixes of
the spacified AFI which were received from the customer
AS.
</t>
</list> </list>
</t> </t>
</section> </section>
@ -978,22 +998,22 @@
particular Address Family. Receipt of an ASPA PDU particular Address Family. Receipt of an ASPA PDU
announcement (announce/withdraw flag == 1) when the router announcement (announce/withdraw flag == 1) when the router
already has an ASPA PDU with the same Customer Autonomous already has an ASPA PDU with the same Customer Autonomous
System Number and the same Address Family (see Flags field), System Number and the same Address Family (see AFI Flags
replaces the previous one. This is to avoid a race condition field), replaces the previous one. This is to avoid a race
when a BGP announcement is received between a withdrawn ASPA condition when a BGP announcement is received between a
PDU and a newly announced ASPA PDU. Therefore, the cache MUST withdrawn ASPA PDU and a newly announced ASPA PDU. Therefore,
deliver the complete data of an ASPA record in a single ASPA the cache MUST deliver the complete data of an ASPA record in
PDU. a single ASPA PDU.
</t> </t>
<t> <t>
The router should see at most one ASPA from a cache for a The router should see at most one ASPA for a given AFI from a
particular Customer Autonomous System Number active at any cache for a particular Customer Autonomous System Number
time. As a number of conditions in the global RPKI may active at any time. As a number of conditions in the global
present multiple valid ASPA RPKI records for a single customer RPKI may present multiple valid ASPA RPKI records for a single
to a particular RP cache, this places a burden on the cache to customer to a particular RP cache, this places a burden on the
form the union of multiple ASPA records it has received from cache to form the union of multiple ASPA records it has
the global RPKI into one ASPA PDU. received from the global RPKI into one ASPA PDU.
</t> </t>
<t> <t>
@ -1004,11 +1024,11 @@
For the ASPA PDU, the announce/withdraw Flag is set to 1 to For the ASPA PDU, the announce/withdraw Flag is set to 1 to
indicate either the announcement of a new ASPA record or a indicate either the announcement of a new ASPA record or a
replacement for a previously announced record with the same replacement for a previously announced record with the same
Customer Autonomous System Number. The announce/withdraw flag Customer Autonomous System Number and AFI. The
set to 0 indicates removal of the ASPA record in total. Here, announce/withdraw flag set to 0 indicates removal of the ASPA
only the customer AS of the ASPA record MUST be provided, the record in total. Here, only the AFI and the customer AS of
Provider AS Count as well as the Provider AS Numbers list MUST the ASPA record MUST be provided, the Provider AS Count as
BE zero. well as the Provider AS Numbers list MUST be zero.
</t> </t>
<t> <t>
@ -1083,7 +1103,7 @@
<list style="hanging"> <list style="hanging">
<t hangText="Minimum allowed value:">1 second.</t> <t hangText="Minimum allowed value:">1 second.</t>
<t hangText="Maximum allowed value:">86400 seconds (1 day).</t> <t hangText="Maximum allowed value:">86400 seconds (1 day).</t>
<t hangText="Recommended default:">3600 seconds (2 hours).</t> <t hangText="Recommended default:">3600 seconds (1 hour).</t>
</list> </list>
</t> </t>
<t hangText="Retry Interval:"> <t hangText="Retry Interval:">
@ -1112,7 +1132,7 @@
<list style="hanging"> <list style="hanging">
<t hangText="Minimum allowed value:">600 seconds (10 minutes).</t> <t hangText="Minimum allowed value:">600 seconds (10 minutes).</t>
<t hangText="Maximum allowed value:">172800 seconds (2 days).</t> <t hangText="Maximum allowed value:">172800 seconds (2 days).</t>
<t hangText="Recommended default:">3600 seconds (1 hour).</t> <t hangText="Recommended default:">7200 seconds (2 hours).</t>
</list> </list>
</t> </t>
</list> </list>
@ -1143,7 +1163,10 @@
Version") and terminate the connection; in which case the Version") and terminate the connection; in which case the
Arbitrary Text field of the ERROR Report PDU MUST be a list of Arbitrary Text field of the ERROR Report PDU MUST be a list of
one octet binary integers indicating the version numbers the one octet binary integers indicating the version numbers the
cache supports. cache supports. The router MUST choose the highest mutally
supported version. If there are none, the router MUST abort the
session, sending a version 2 Error Report PDU with Error Code 4
("Unsupported Protocol Version").
</t> </t>
<t> <t>
If a router which supports version N sends a query to a cache If a router which supports version N sends a query to a cache
@ -1152,9 +1175,9 @@
<list style="numbers"> <list style="numbers">
<t> <t>
The cache may terminate the connection, perhaps with a The cache may terminate the connection, perhaps with a
version 4 Error Report PDU, Unsupported Protocol Version. version 2 Error Report PDU with Error Code 4 ("Unsupported
In this case, the router MAY retry the connection using Protocol Version"). In this case, the router MAY retry the
protocol version C. connection using protocol version C.
</t> </t>
<t> <t>
The cache may reply with a version C response. In this The cache may reply with a version C response. In this