MIME-Version: 1.0
X-Mailer: MIME-tools 5.508 (Entity 5.508)
X-RT-Interface: Web
Content-Transfer-Encoding: binary
Content-Disposition: inline
References: <rt-4.2.8-62213-1485427934-1350.0-0-0@isc.org>
 <RT-Ticket-44535@isc.org>
Message-ID: <rt-4.4.1-99844-1513770344-140.44535-0-0@isc.org>
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
In-Reply-To: <rt-4.2.8-62213-1485427934-1350.0-0-0@isc.org>
RT-Send-CC:
Content-Length: 618

Comments about the I-D (10.txt):
 - RAI sub-options have no real official names,
  IMHO in SUBOPT_RELAY_PORT the SUBOPT_
  is not necessary but it does not matter.
  BTW the code uses also things around relay-port.

 - I have more a concern about
   OPTION_RELAY_RELAY_PORT which should be
   OPTION_RELAY_SOURCE_PORT. IMHO it is a typo.

 - in 5.2 in the server behavior I don't understand
  what "check" means in "the IPv6 server MUST
  check and use the UDP source port from the UDP
  packet..."?

To conclude about the I-D I use as temporary values
19 (next free value) and 135 (vs 90 which is already
assigned).