From ea02eaee318694488329d1a3f80a39939c677d2e Mon Sep 17 00:00:00 2001
From: Randy Bush <randy@psg.com>
Date: Sun, 13 Feb 2022 14:23:27 -0800
Subject: [PATCH] some things from keyur review

---
 draft-ymbk-idr-l3-discovery.xml | 153 ++++++++++++++++++--------------
 1 file changed, 84 insertions(+), 69 deletions(-)

diff --git a/draft-ymbk-idr-l3-discovery.xml b/draft-ymbk-idr-l3-discovery.xml
index db3267b..14fe364 100644
--- a/draft-ymbk-idr-l3-discovery.xml
+++ b/draft-ymbk-idr-l3-discovery.xml
@@ -31,21 +31,6 @@
       </address>
     </author>
 
-    <author fullname="Susan Hares" initials="S" surname="Hares">
-      <organization>Hickory Hill Consulting</organization>
-      <address>
-        <postal>
-          <street>7453 Hickory Hill</street>
-          <city>Saline</city>
-          <region>MI</region>
-          <code>48176</code>
-          <country>USA</country>
-        </postal>
-          <phone>+1-734-604-0332</phone>
-        <email>shares@ndzh.com</email>
-      </address>
-    </author>
-      
     <author fullname="Russ Housley" initials="R" surname="Housley">
       <organization abbrev="Vigil Security">Vigil Security, LLC</organization>
       <address>
@@ -67,6 +52,21 @@
       </address>
     </author>
     
+    <author fullname="Susan Hares" initials="S" surname="Hares">
+      <organization>Hickory Hill Consulting</organization>
+      <address>
+        <postal>
+          <street>7453 Hickory Hill</street>
+          <city>Saline</city>
+          <region>MI</region>
+          <code>48176</code>
+          <country>USA</country>
+        </postal>
+          <phone>+1-734-604-0332</phone>
+        <email>shares@ndzh.com</email>
+      </address>
+    </author>
+      
     <author fullname="Keyur Patel" initials="K." surname="Patel">
       <organization>Arrcus</organization>
       <address>
@@ -456,14 +456,16 @@ Flags (bit):
         </artwork>
       </figure>
 
+    <t>The Payload Length is 24 to cover the Flags and Port fields.</t>
+
     <t>The Port is the TCP Port Number (TBD3) on which the HELLO sender
     MUST have a waiting TLS/TCP (as specified in Flags) server
     listening.  Though the IANA assigned well-known port SHOULD be used,
     this field allows configuration of alternate ports.</t>
       
     <t>The HELLO PDU is unique in that it is encapsulated in a multicast
-    UDP packet.  It solicits TLS/TCP connections from other peers on a
-    link.  See <xref target="dhello"/> for why multicast is used.</t>
+    UDP packet.  It offers TLS/TCP service to peers on a link.  See
+    <xref target="dhello"/> for why multicast is used.</t>
 
     <t>The HELLO PDU is an IPv4 or IPv6 UDP PDU sent to a link-local
     multicast address.  It solicits a unicast TLS/TCP open request(s) of
@@ -477,7 +479,8 @@ Flags (bit):
     peer's destination IP address is known after the HELLO.</t>
 
     <t>When an interface is turned up on a device, it SHOULD issue a
-    HELLO if it is to participate in L3ND sessions.</t>
+    HELLO if it is to participate in L3ND sessions and repeat HELLOs at
+    a configured interval, with a default of 60 seconds.</t>
 
     <!-- 
     <t>If a constrained Nearest Bridge destination address has been
@@ -514,7 +517,8 @@ Flags (bit):
     <t>If a HELLO is received from an IP Address with which there is an
     established session for that AFI, the HELLO should be dropped.</t>
 
-    <t>The Payload Length is 24 to cover the Flags and Port fields.</t>
+    <t>A device with a TLS/TCP listener SHOULD log or otherwise report
+    repeated failed inbound attempts.</t>
 
     </section>
 
@@ -525,6 +529,10 @@ Flags (bit):
     Self-Signed or CA-based, the receiver SHOULD respond with a HELLO
     specifying its preferences.</t>
 
+    <t>As it is assumed that the configured deployment of a data center
+    would have compatible parameters on all devices, any disagreement
+    over TLS/TCP or trust anchors MUST be logged.</t>
+
     <t>If the receiver of a HELLO agrees with the sender's choice of
     TLS/TCP and authentication, both sides have agreed on an AFI for the
     transport and on each other's IP address in that AFI.  This is
@@ -548,6 +556,11 @@ Flags (bit):
     <t>If the TLS/TCP open fails, then this SHOULD be logged and the
     parties MUST go back to the initial state and try HELLO.</t>
 
+    <t>Should an interface with an established TLS/TCP session be
+    reconfigured changing the TLS/TCP parameters, the TLS/TCP session
+    should be torn down and the devices should restart/resume
+    HELLOs.</t>
+
     <t>Once the TLS/TCP session has been established, the two devices
     exchange L3ND PDUs, starting with OPENs.</t>
 
@@ -571,13 +584,13 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 2 |         Payload Length        |
+|  Version = 0  |  PDU Type = 2 |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Nonce             |
+~                               |             Nonce             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |   AttrCount   |               |
+~                               |   AttrCount   |               ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+               +
-|                       Attribute List ...                      |
+~                       Attribute List ...                      ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
 |                         Serial Number                         |
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
@@ -672,11 +685,11 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 3 |       Payload Length = 5      |
+|  Version = 0  |  PDU Type = 3 |       Payload Length = 5      ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |   ACKed PDU   | EType |       |
+~                               |   ACKed PDU   | EType |       ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|   Error Code  |           Error Hint          |
+~   Error Code  |           Error Hint          |
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -783,13 +796,13 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |    PDU Type   |         Payload Length        |
+|  Version = 0  |    PDU Type   |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Count             |
+~                               |             Count             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |                 Serial Number                 |
+~               |                 Serial Number                 ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |             Encapsulation List...             |
+~               |             Encapsulation List...             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -895,15 +908,15 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 4 |         Payload Length        |
+|  Version = 0  |  PDU Type = 4 |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Count             |
+~                               |             Count             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |                 Serial Number                 |
+~               |                 Serial Number                 ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |  Encaps Flags |          IPv4 Address         |
+~               |  Encaps Flags |          IPv4 Address         ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |   PrefixLen   |    more ...   |
+~                               |   PrefixLen   |    more ...   ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -929,21 +942,21 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 5 |         Payload Length        |
+|  Version = 0  |  PDU Type = 5 |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Count             |
+~                               |             Count             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |                 Serial Number                 |
+~               |                 Serial Number                 ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |  Encaps Flags |                               |
+~               |  Encaps Flags |                               ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+                               +
-|                                                               |
+|                                                               ~
 +                                                               +
-|                                                               |
+~                                                               ~
 +                                                               +
-|                          IPv6 Prefix                          |
+~                          IPv6 Prefix                          ~
 +                               +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |   PrefixLen   |    more ...   |
+~                               |   PrefixLen   |    more ...   ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
           </artwork>
         </figure>
@@ -971,7 +984,7 @@ Flags (bit):
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
 |  Label Count  |                 Label                 | Exp |S|
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                 Label                 | Exp |S|    more ...   |
+~                 Label                 | Exp |S|    more ...   ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -998,17 +1011,17 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 6 |         Payload Length        |
+|  Version = 0  |  PDU Type = 6 |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Count             |
+~                               |             Count             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |                 Serial Number                 |
+~               |                 Serial Number                 ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |  Encaps Flags |      MPLS Label List ...      |
+~               |  Encaps Flags |      MPLS Label List ...      ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
 |                          IPv4 Address                         |
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|   PrefixLen   |                    more ...                   |
+|   PrefixLen   |                    more ...                   ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -1034,23 +1047,23 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  |  PDU Type = 7 |         Payload Length        |
+|  Version = 0  |  PDU Type = 7 |         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |             Count             |
+~                               |             Count             ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |                 Serial Number                 |
+~               |                 Serial Number                 ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |  Encaps Flags |      MPLS Label List ...      |
+~               |  Encaps Flags |      MPLS Label List ...      |
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                                                               |
+|                                                               ~
 +                                                               +
-|                                                               |
+~                                                               ~
 +                          IPv6 Address                         +
-|                                                               |
+~                                                               ~
 +                                                               +
-|                                                               |
+~                                                               |
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|   Prefix Len  |                    more ...                   |
+|   Prefix Len  |                    more ...                   ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -1072,13 +1085,13 @@ Flags (bit):
  0                   1                   2                   3  
  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|  Version = 0  | PDU Type = 255|         Payload Length        |
+|  Version = 0  | PDU Type = 255|         Payload Length        ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |         Serial Number         |
+~                               |         Serial Number         ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|                               |       Enterprise Number       |
+~                               |       Enterprise Number       ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
-|               |    Ent Type   |      Enterprise Data ...      |
+~               |    Ent Type   |      Enterprise Data ...      ~
 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         </artwork>
       </figure>
@@ -1104,14 +1117,16 @@ Flags (bit):
 
     <section anchor="dhello" title="HELLO Discussion">
 
-      <t>A device may send IP packets an L3 interface which transmit
+      <t>A device may send IP packets an L3 interface which transmits
       data over a single Layer-2 (L2) interface or multiple L2
       interfaces.  Packets sourced by one L3 IP interface over multiple
       L2 should consider that an L3 interface with multiple Layer-2
-      interfaces may come at various times, therefore the HELLO PDU
-      configuration time SHOULD be set to a non-zero value.  Packets
-      transmitted on a single L2 interface on a point-to-point (p2p)
-      connection, MAY set the configuration value to zero.</t>
+      interfaces could have many devices which might come at various
+      times, therefore the configured HELLO PDU retransmit time SHOULD
+      be set to a non-zero value, and periodic HELLOs should continue.
+      Packets transmitted on a single L2 interface on a point-to-point
+      (p2p) connection, MAY set the configuration value to zero, so once
+      a TLS/TCP session is up, HELLOs are no longer desirable.</t>
 
       <t>A device with multiple Layer-2 interfaces, traditionally called
       a switch, may be used to forward frames and therefore packets from
@@ -1144,12 +1159,12 @@ Flags (bit):
 
     <t>An implementation SHOULD provide the ability to configure each
     logical interface as L3ND speaking or not.</t>
-
+<!--
     <t>An implementation SHOULD provide the ability to configure whether
     HELLOs on an L3ND enabled interface send Nearest Bridge or the MAC
     which is propagated by switches from that interface; see <xref
     target="hello"/>.</t>
-
+-->
     <t>An implementation SHOULD provide the ability to distribute one or
     more loopback addresses or interfaces into L3ND on an external L3ND
     speaking interface.</t>