Re: [bfcpbis] Updates to draft-ietf-bfcpbis-rfc4583bis required to enable ICE

"Charles Eckel (eckelcu)" <eckelcu@cisco.com> Thu, 06 April 2017 19:26 UTC

Return-Path: <eckelcu@cisco.com>
X-Original-To: bfcpbis@ietfa.amsl.com
Delivered-To: bfcpbis@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6B87E12962F for <bfcpbis@ietfa.amsl.com>; Thu, 6 Apr 2017 12:26:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.521
X-Spam-Level:
X-Spam-Status: No, score=-14.521 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 9VCF7_54zbI6 for <bfcpbis@ietfa.amsl.com>; Thu, 6 Apr 2017 12:26:44 -0700 (PDT)
Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 677AF129632 for <bfcpbis@ietf.org>; Thu, 6 Apr 2017 12:26:43 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=12526; q=dns/txt; s=iport; t=1491506803; x=1492716403; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=GWHB/m9AbCV4FPIBSFn7yI5QkmSbbR+ptuGayU0tvkI=; b=JWJVFO5G0zm69sRi8vieTrOnPJorGFdin4hyg4WfSOAGLPU4xgL798We Q2Fg+7rKo7q3vjchxil4t9DqMIfNSKioc7lNy5TC4bVjnsUuwncwUu4uT WhEZspYbs8DlaScp5bdxwfYtf0sEegA7R0f/wMux2zX/mOaMftEQMywV+ A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0ATAgBgleZY/5BdJa1cGQEBAQEBAQEBAQEBBwEBAQEBgm47K2GBCweDXIoSkT6IGogHhTSCD4YiAhqDLD8YAQIBAQEBAQEBayiFFQEBAQEDI1YQAgEIDgMDAQIoAwICAh8RFAkIAgQBDQWJdgMVqh6CJiuHBQ2DNAEBAQEBAQEBAQEBAQEBAQEBAQEBAR2IU4FigQmCUYIMGRaCUC6CMQWWHYYbOwGOGYQ5gX6FLooRiF6CHYh8AR84gQVbFVIBhkl1AYhRgQ0BAQE
X-IronPort-AV: E=Sophos;i="5.37,160,1488844800"; d="scan'208,217";a="229779898"
Received: from rcdn-core-8.cisco.com ([173.37.93.144]) by rcdn-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Apr 2017 19:26:42 +0000
Received: from XCH-RCD-012.cisco.com (xch-rcd-012.cisco.com [173.37.102.22]) by rcdn-core-8.cisco.com (8.14.5/8.14.5) with ESMTP id v36JQgjb030054 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 6 Apr 2017 19:26:42 GMT
Received: from xch-aln-018.cisco.com (173.36.7.28) by XCH-RCD-012.cisco.com (173.37.102.22) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Thu, 6 Apr 2017 14:26:41 -0500
Received: from xch-aln-018.cisco.com ([173.36.7.28]) by XCH-ALN-018.cisco.com ([173.36.7.28]) with mapi id 15.00.1210.000; Thu, 6 Apr 2017 14:26:41 -0500
From: "Charles Eckel (eckelcu)" <eckelcu@cisco.com>
To: Roman Shpount <rshpount@turbobridge.com>, Tom Kristensen <2mkristensen@gmail.com>
CC: "Tom Kristensen (tomkrist)" <tomkrist@cisco.com>, Tom Kristensen <tomkri@ifi.uio.no>, "bfcpbis@ietf.org" <bfcpbis@ietf.org>, Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>, "Paul E. Jones" <paulej@packetizer.com>, Christer Holmberg <christer.holmberg@ericsson.com>, Mary Barnes <mary.ietf.barnes@gmail.com>
Thread-Topic: [bfcpbis] Updates to draft-ietf-bfcpbis-rfc4583bis required to enable ICE
Thread-Index: AQHSk6JSUjB4HtjJ8UWHVVBUVasJT6G4+RwAgAA3TgD//545gA==
Date: Thu, 06 Apr 2017 19:26:41 +0000
Message-ID: <C5B91F83-66D4-47E7-A94D-DFE483583F5B@cisco.com>
References: <CAD5OKxs9NN1CtNYaZEiGUxK-UUs=LwYq=A8n69LZ4REE80EzUQ@mail.gmail.com> <CAFHv=r82b6s8A5=Z33OGcnZN5My0ArpFwzJPxpbs+wwiu7wgBA@mail.gmail.com> <CAD5OKxsaP4UC+feoDTVRHBk=0TOKUXbb989hL_=v1-auHRuE0A@mail.gmail.com>
In-Reply-To: <CAD5OKxsaP4UC+feoDTVRHBk=0TOKUXbb989hL_=v1-auHRuE0A@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/f.20.0.170309
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.24.90.52]
Content-Type: multipart/alternative; boundary="_000_C5B91F8366D447E7A94DDFE483583F5Bciscocom_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/bfcpbis/DArPZYzmj49FdBDfDt8Z_74CQAc>
Subject: Re: [bfcpbis] Updates to draft-ietf-bfcpbis-rfc4583bis required to enable ICE
X-BeenThere: bfcpbis@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: BFCPBIS working group discussion list <bfcpbis.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bfcpbis/>
List-Post: <mailto:bfcpbis@ietf.org>
List-Help: <mailto:bfcpbis-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 06 Apr 2017 19:26:47 -0000

Please see inline with [cue].

From: Roman Shpount <rshpount@turbobridge.com>
Date: Thursday, April 6, 2017 at 11:16 AM
To: Tom Kristensen <2mkristensen@gmail.com>
Cc: Charles Eckel <eckelcu@cisco.com>, Tom Kristensen <tomkrist@cisco.com>, Tom Kristensen <tomkri@ifi.uio.no>, "bfcpbis@ietf.org" <bfcpbis@ietf.org>, Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>, Paul Jones <paulej@packetizer.com>, Christer Holmberg <christer.holmberg@ericsson.com>, Mary Barnes <mary.ietf.barnes@gmail.com>
Subject: Re: [bfcpbis] Updates to draft-ietf-bfcpbis-rfc4583bis required to enable ICE

On Thu, Apr 6, 2017 at 10:58 AM, Tom Kristensen <2mkristensen@gmail.com<mailto:2mkristensen@gmail.com>> wrote:
On 2 March 2017 at 23:14, Roman Shpount <rshpount@turbobridge.com<mailto:rshpount@turbobridge.com>> wrote:
1. In Section 3, we need to define an additional proto field value: TCP/DTLS/BFCP, which is realized by running BFCP on top of DTLS as descibed in this specification and running DTLS on top of TCP is realized using the framing method defined in RFC4571, with DTLS packets being sent and received instead of RTP/RTCP packets using the shim defined in RFC4571 so that length field defined in RFC4571 precedes each DTLS message. All sections of the document should be updated to include the TCP/DTLS/BFCP transport.

This was questioned by Charles and I agree with the conclusion, backwards and forward compliancy is good although confusing/differing "syntax".

I am not sure if this means you want to change this somehow or that you are going to update the draft with TCP/DTLS/BFCP. I agree that the syntax is not ideal but it is in line with other documents such as dtls-sdp or sctp-sdp.

[cue] I believe this was meant to express agreement with the consensus that while not perfect, adding TCP/DTLS/BFCP to the draft is the best of the alternatives explored.

Cheers,
Charles

The id-nits checker made me aware of RFC 8122 obsoleting RFC 4572, I reckon the additions necessary from the draft-ietf-mmusic-4572-update is covered in the new RFC 8122 or not?

draft-ietf-mmusic-4572-update is now published as RFC 8122, so references to RFC 4572 should be updated to RFC 8122

The rest sounds good.

When you publish the updated version I will review and provide comments. I will also sync comments which were made against the ICE Considerations section of sctp-sdp with your draft.

Regards,
_____________
Roman Shpount