Hi! In reviewing the corresponding YANG module for this document, I was reminded that IP-TFS SA are unidirectional. In principle it would be possible to setup an IP-TFS tunnel for only one side of a connection. Section 8 should mention that risk that this introduces.
Regards, Roman > -----Original Message----- > From: Roman Danyliw > Sent: Wednesday, May 4, 2022 5:20 PM > To: ipsec@ietf.org WG <ipsec@ietf.org> > Subject: AD Review of draft-ietf-ipsecme-iptfs-12 > > Hi! > > I performed an AD review of draft-ietf-ipsecme-iptfs-12. Thank you for this > work and the patience of the WG in getting it processed. > > I have a number of comments below, but the document is in good shape so > please process them concurrently with the IETF LC feedback. > > ** Thank you for getting an early TSVART review and addressing that feedback > given congestion control issues this work raises > > ** Section 1. Editorial. > > OLD > While directly > obscuring the data with encryption [RFC4303], the traffic pattern > itself exposes information due to variations in its shape and timing > > NEW > While directly obscuring the data with encryption [RFC4303], the patterns in > the message traffic may exposes information due to variations in its shape and > timing. > > ** Section 2.1. Typo. s/the the/the/ > > ** Section 2.4.2.1. > > Enabling circuit breakers is also a reason a user > may wish to enable congestion information reports ... > > Consider if s/a user may wish to/local policy may/ to generalize who is doing > the tuning. There are a few other places were "user" is the noun tuning the > stack. > > ** Section 3. > > Thus, to support > congestion control the receiver must have a paired SA back to the > sender > > Should the be a "MUST" (i.e., s/receiver must have/received MUST have)? > > ** Section 3. > > If the SA back to the sender is a non-AGGFRAG_PAYLOAD enabled SA then an > AGGFRAG_PAYLOAD empty payload (i.e., header only) > is used to convey the information. > > I'm missing something -- if the SA is not AGGFRAG_PAYLOAD-enabled, what > how can it send anything AGGFRAG related? > > ** Section 4. > > All IP-TFS specific configuration should be specified > at the unidirectional tunnel ingress (sending) side > > Should is used here. Not in the normative sense. What would be the > alternative to specifying the unidirectional tunnels on the sending side? > > ** Section 4.1. > > For non-congestion > controlled mode, the bandwidth SHOULD be configured. > > What would it mean for this not be configured? That the end-point set a > packet size and rate? > > ** Section 4.3. Is this a generic statement that how congestion control is > done > is a local configuration option, or is this a Boolean configuration of use > congestion control or not (aka, Section 6.1.1 vs. 6.1.2.?) > > ** Section 5.1. > > If any > requirement flags are not understood or cannot be supported by the > receiver then the receiver SHOULD NOT enable use of AGGFRAG_PAYLOAD > > Is the WG sure that this shouldn't be MUST NOT? What's the user case where > you want to continue? > > ** Section 6.1.4. Typo. s/it's USE_AGGFRAG/its USE_AGGFRAG/ > > ** Section 7.1. Why the reference to RFC7120? Since this registry is going > to > be "Expert Review", this document doesn't apply. > > ** Section 7.1. To double check, there is no more guidance to provide to the > expert reviewer? > > ** Section 8. Editorial. > > This document describes an aggregation and fragmentation mechanism > and it use to add TFC to IP traffic. The use described is expected > to increase the security of the traffic being transported. > > The first sentence doesn't parse and I recommend more precision on the > second. Perhaps: > > NEW > > This document describes an aggregation and fragmentation mechanism to > efficiently implement TFC for IP traffic. This approach is expected to reduce > the > efficacy of traffic analysis on IPSec communication. > > ** Section 8. Editorial. s/As noted in (Section 3.1)/As noted in Section 3.1,/ > > ** Section 8. > > As noted previously in Section 2.4.2, for TFC to be fully maintained ... > > What does it mean to for TFC to be "fully maintained"? > > Regards, > Roman _______________________________________________ IPsec mailing list IPsec@ietf.org https://www.ietf.org/mailman/listinfo/ipsec