Re: [quicwg/base-drafts] Scope of disable_active_migration (#3670)

[Kazuho Oku <[e-mail]>]

@kazuho commented on this pull request.

LGTM modulo the point below.

> @@ -4872,11 +4874,11 @@ max_ack_delay (0x0b):
 disable_active_migration (0x0c):
 
 : The disable active migration transport parameter is included if the endpoint
-  does not support active connection migration ({{migration}}). Peers of an
-  endpoint that sets this transport parameter MUST NOT send any packets,
-  including probing packets ({{probing}}), from a local address or port other
-  than that used to perform the handshake.  This parameter is a zero-length
-  value.
+  does not support active connection migration ({{migration}}) on the address
+  being used during the handshake.  Peers of an endpoint that sets this
+  transport parameter MUST NOT send any packets, including probing packets
+  ({{probing}}), from a local address or port other than that used to perform
+  the handshake. This parameter is a zero-length value.

To me it seems that we need to also change this sentence (starting from "Peers ..."). We need to tighten the restriction to only when the client sends the packets to the original address of the server.

Maybe something like: _Peers of an endpoint that sets this transport parameter MUST NOT send any packets, including probing packets ({{probing}}) __to the address that was used to perform the handshake__, from a local address or port other than that used to perform the handshake._

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/3670#pullrequestreview-414895639