Re: [tram] draft-ietf-tram-stun-pmtud-06 review

["Asveren, Tolga" <[e-mail]>]

Hi Marc,

Please see inline for comments/answers/questions.

Thanks,
Tolga

-----Original Message-----
From: Marc Petit-Huguenin <[e-mail]> 
Sent: Saturday, March 3, 2018 10:42 AM
To: Asveren, Tolga <[e-mail]>; Gonzalo Salgueiro ([e-mail]) <[e-mail]>; [e-mail]
Subject: Re: [tram] draft-ietf-tram-stun-pmtud-06 review

Thanks for the review, please see inline.

As the cutoff is Monday afternoon, I'll publish -07 just after sending this email.

On 02/17/2018 09:40 PM, Asveren, Tolga wrote:
> Marc, Gonzalo,
> 
> Some comments/questions for draft-ietf-tram-stun-pmtud-06:
> 
> i- 4.
> 
>   Implementations supporting this specification MUST implement the
>    server side of both the Simple Probing mechanism (Section 4.1) and
>    the Complete Probing mechanism (Section 4.2).
>    Implementations supporting this specification MUST implement the
>    client side of the Complete Probing mechanism.  They MAY implement
>    the client side of the Simple Probing mechanism.
> 
> I don't see the rationale behind mandating this behavior. Such semantics in general are taken care of in the context of a capability negotiation mechanism, which this document does not specify. As it stands right now, I think this is best left to local policy and bilateral agreements and I would suggest not to use anything stronger than "RECOMMEND".

It took me some time to remember why that was the case.  IIRC, one of the reasons of doing this is because of the implicit probe support in section 5.2.  Because there is no explicit signaling there, we need the server side (which was previously the client side) to be ready to respond to both Simple and Complete Probing.  Hence the "server must do both, client choose what they want to do" rule.

The other reason is that if we let implementers choose, then Complete Probing - which is the correct way of doing probing - will never be used.
[TOLGA] If client does not get replies for a mechanism it SHOULD try the other one. Honestly I don’t see a compelling reason to mandate anything on the server side. There is no negotiation mechanism and that can’t be enforced on protocol level anyhow. I suggest to use RECOMMNED for server and add a sentence for client behavior that it SHOULD try the other mechanism if one does not get any replies at all.

> 
> ii- 4.1
> DF bit set over UDP.
> ==>
> DF bit in IP Header.
> Right?  (the same in 4.2)

Not really, the Probe Request is still sent over UDP, so I changed the text to that (twice):

 The Simple Probing mechanism is implemented by sending a Probe  Request with a PADDING [RFC5780] attribute over UDP with the DF bit  set in the IP header.  A router on the path to the server can reject  this request with an ICMP message or drop it.

 The Complete Probing mechanism is implemented by sending one or more  Probe Indications with a PADDING attribute over UDP with the DF bit  set in the IP header followed by a Report Request to the same server.
[TOLGA] Looks good, I just wanted to make sure about the "DF bit in IP header" part.

> 
> iii- 4.1.1
> A client MUST NOT send a probe if it does not have knowledge that the
>    server supports this specification.  This is done either by external
>    signalling or by a mechanism specific to the UDP protocol to which
>    PMTUD capabilities are added or by one of the mechanisms specified in
>    Section 5.
> I suggest adding text for possible use of local policy/bilateral agreements for this purpose.

I am not sure what to add to what is said.  Do you have text to suggest?
[TOLGA] Maybe by just adding something like:
"Information about client/server support for this specification, for example, could be based on local policy and bilateral agreements among relevant organizations."

> 
> iv-  4.2
> Some protocols may already have a way of identifying each individual
>    UDP packet, in which case these identifiers SHOULD be used in the
>    IDENTIFIERS attribute of the Report Response.
> What is meant here as "some protocols"? Are you referring  to what is carried as payload of UDP or what is used below UDP?

The former.  I changed the text to "application layer protocols:"
[TOLGA] Thanks.

> 
> v- 4.2
> In the first packet identification mechanism, the server computes a
>    checksum over each packet received and sends back to the sender the
>    list of checksums ordered chronologically.  The client compares this
>    list to its own list of checksums.
> What if there are two probes with the same checksum right after each other and one of them is lost? How can client disambiguate such a case in  the response?

I am not sure to get the problem here.  The sender keeps a chronologically ordered list of the checksums of the sent packets (data and probes), so in that case there is a sequence of two identical checksums in that list.  The server sends back a list of checksums, but with only one of that value (or even 0).  The loss can be detected.
[TOLGA] Loss can be detected but which one is lost?

Probe-n  (checksum-A)
Probe-n+1 (checksum-A)

Only a single reply for (checksum-A) is received.

Is Probe-n or Probe-n+1 lost?

> 
> v- 7
> The Simple Probing mechanism may be used without authentication
>    because this usage by itself cannot trigger an amplification attack
>    because the Probe Response is smaller than the Probe Request.
> Isn't the same true for Sum(Probe Requests) > (Report Response)?

I think that mechanism was set following a review that was worried about amplification attack.  We have the simple probe mechanism without authentication to keep the barrier to entry very low, i.e. the minimum required implementation possible.

Security expert can chime in.
[TOLGA] I think that would be good. Who would be that?

> 
> vii- nit
> == The copyright year in the IETF Trust and authors Copyright Line does not
>      match the current year
> 
>   -- The document date (September 1, 2017) is 169 days in the past.  Is this
>      intentional?

Will be upgraded with the next publication.
[TOLGA] Thanks.

--
Marc Petit-Huguenin
Email: [e-mail]
Blog: https://marc.petit-huguenin.org
Profile: https://www.linkedin.com/in/petithug