Re: [Tsv-art] [tram] Tsvart last call review of draft-ietf-tram-turnbis-25

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, all,

Here are my key points:

- whether you deal with TCP options or simply ignore those on incoming connections and use defaults on outgoing, the issue needs to be addressed - as well as its impact on your use.
- UDP options are not experimental; they’re standards-track; it could be mentioned here non-normatively as a possible workaround to the IP fragmentation issue.

Finally, IMO, assuming that RTP/RTCP can or should provide extended functions that might already be provided by existing TCP options or emerging UDP options seems like both wasted effort and a failed opportunity to tune the transport as it was intended.

Overall, the point is that simply ignoring this issue is insufficient - at a minimum, this doc needs to clearly state that this issue is being ignored and why, as well as addressing that issue in the security sections.

Finally, I’m quite bothered by the glib idea that transport packets can simply be translated into each other either between two TCP connections or between TCP and UDP. The notion is simply false; TCP doesn’t preserve message boundaries and TCP segments are not guaranteed to match application message boundaries. 

I.e., the transport implications of this “hack” are inadequately addressed.

Joe

On Jun 11, 2019, at 1:28 AM, Magnus Westerlund <magnus.westerlund@xxxxxxxxxxxx> wrote:

Hi Joe and Tiru,

May I hazard a guess why this have not arisen is that there are no transport protocol options that makes sense to use end-to-end and are not protocol specific. Thus, in UDP <-> TCP translations by TURN server there has so far not been a need to carry any of them over. Joe, can you think of any that would make sense?

For UDP <-> UDP the experimental proposal for UDP options I don't see that we can require this specification to have to specify that. I do think it is an interesting question for https://datatracker.ietf.org/doc/draft-ietf-tsvwg-udp-options/ if that should write more about what to do with the options when performing translation operations?

When it comes to RTP and RTCP that is widely used over TURN relays when those applications need extended functionality they have gone ahead and extended RTP/RTCP rather than attempting to affect lower layers where other entities than the end-points are required to be upgraded.

Cheers

Magnus




On 2019-06-11 07:20, Konda, Tirumaleswar Reddy wrote:

Hi Joe,

 

I meant the specifications that use TURN (ICE, SIP and WebRTC) do not discuss setting any TCP option for application data (RTP, RTCP and WebRTC data channels).  Please note TCP is only used as fallback transport only if UDP traffic is blocked to the TURN server.

TURN has been widely deployed in the field, and there was no discussion in the WG to explicitly handle TCP options.

 

Cheers,

-Tiru

 

From: Joe Touch <touch@xxxxxxxxxxxxxx>
Sent: Monday, June 10, 2019 7:59 PM
To: Konda, Tirumaleswar Reddy <TirumaleswarReddy_Konda@xxxxxxxxxx>
Cc: tsv-art@xxxxxxxx; draft-ietf-tram-turnbis.all@xxxxxxxx; ietf@xxxxxxxx; tram@xxxxxxxx
Subject: Re: [Tsv-art] [tram] Tsvart last call review of draft-ietf-tram-turnbis-25

 

CAUTION: External email. Do not click links or open attachments unless you recognize the sender and know the content is safe.


Hi, Tiru,



On Jun 9, 2019, at 11:43 PM, Konda, Tirumaleswar Reddy <TirumaleswarReddy_Konda@xxxxxxxxxx> wrote:

 

On Jun 7, 2019, at 4:39 AM, Konda, Tirumaleswar Reddy
<TirumaleswarReddy_Konda@xxxxxxxxxx> wrote:


The specification has two sections 14 and 15 (IP Header Fields for
UDP-to-

UDP translation and IP Header Fields for TCP-to-UDP translation) to
discuss direct translations. https://tools.ietf.org/html/rfc5766 only
covered UDP-to- UDP translation in Section 12.

Yes, but both sections ignore the impact of transport options - both
current for TCP and pending for UDP. These are ignored both when
packets with such transport options are received (the input packet to
the translation) and whether / how they are used on transmit (the
output packet)


TURN is used to relay real-time data (e.g. audio and video streams)
and the approach taken by VOIP related specifications is to avoid
fragmentation for RTP packets


Sec 2.8 mentions RTP as one use case envisioned (at this point, it’d be fair to
ask this revision to clarify whether that turned out to be true). But it isn’t
indicated as the only use case.


The draft says TURN is invented to support multimedia sessions signaled using SIP and is typically used with ICE. TURN is also used with WebRTC, and WebRTC data channels also 
avoid IP fragmentation (see 
https://tools.ietf.org/html/draft-ietf-rtcweb-data-channel-13). 

 

The application protocols TURN is designed for or typically used for is not relevant to my point above, unless you’re claiming that these uses never use transport options (which is doubtful for TCP, for which some transport options are pervasively used by default).






Regardless, though, this doesn’t impact the concern raised above. RTP could
still employ transport options.


I checked again and don't see any RTP, Back-to-Back User Agents (B2BUAs), SIP proxies and WebRTC gateway specifications discussing transport options for translations.

 

The fact that others have this gap does not justify continuing to fail to address it in this document. If anything, it makes it that much more important to address.

 

Joe


-- 

Magnus Westerlund 

----------------------------------------------------------------------
Network Architecture & Protocols, Ericsson Research
----------------------------------------------------------------------
Ericsson AB                 | Phone  +46 10 7148287
Torshamnsgatan 23           | Mobile +46 73 0949079
SE-164 80 Stockholm, Sweden | mailto: magnus.westerlund@xxxxxxxxxxxx
----------------------------------------------------------------------
_______________________________________________
Tsv-art mailing list
Tsv-art@xxxxxxxx
https://www.ietf.org/mailman/listinfo/tsv-art


[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux