Re: [Sipbrandy] AD Evaluation of draft-ietf-sipbrandy-osrtp-07

Ben Campbell <ben@nostrum.com> Tue, 26 March 2019 16:10 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: sipbrandy@ietfa.amsl.com
Delivered-To: sipbrandy@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 94FCD1205D2; Tue, 26 Mar 2019 09:10:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.679
X-Spam-Level:
X-Spam-Status: No, score=-1.679 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_INVALID=0.1, DKIM_SIGNED=0.1, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=fail (1024-bit key) reason="fail (message has been altered)" header.d=nostrum.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 En8rywrr1G6p; Tue, 26 Mar 2019 09:10:16 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 291951205B0; Tue, 26 Mar 2019 09:10:14 -0700 (PDT)
Received: from dhcp-836a.meeting.ietf.org (dhcp-836a.meeting.ietf.org [31.133.131.106]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id x2QGA58Y073327 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Tue, 26 Mar 2019 11:10:07 -0500 (CDT) (envelope-from ben@nostrum.com)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=nostrum.com; s=default; t=1553616609; bh=c6D64qscARfDgK+eFQO+nY6ui1OEqL0095fI9pHkk/E=; h=From:Subject:Date:In-Reply-To:Cc:To:References; b=j1v13hvJ1Bsgj9ILLccbmdJ+DrZ3shX9542H861vlXBdY9VyeeD9Ac7aQUbTYOLZ9 21bMog+XqEpls/xI6QRgVz0qUoFby/Of/BVG4ZjyEtwq8y6bcSYVp+VkLKlVL6jYvu hHyUs0yacgz1WgN1wKVXsocTdqWmqKg/qYIFO374=
From: Ben Campbell <ben@nostrum.com>
Message-Id: <A7A08115-5B69-4931-8C89-0EBDF3A76D10@nostrum.com>
Content-Type: multipart/signed; boundary="Apple-Mail=_E52DE8B7-9E99-408D-9D66-D98EB136A8F0"; protocol="application/pgp-signature"; micalg="pgp-sha512"
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
Date: Tue, 26 Mar 2019 17:10:00 +0100
In-Reply-To: <CAB7PXwRSFXcB5zGdNP_zqyKWUJqZAK+bKsxeyWhK6eeogqJ8dw@mail.gmail.com>
Cc: Gonzalo Camarillo <gonzalo.camarillo@ericsson.com>, "draft-ietf-sipbrandy-osrtp.all@ietf.org" <draft-ietf-sipbrandy-osrtp.all@ietf.org>, "sipbrandy@ietf.org" <sipbrandy@ietf.org>, Alexey Melnikov <Alexey.Melnikov@isode.com>
To: Andy Hutton <andyhutton.ietf@gmail.com>
References: <72C42C63-D5C4-403D-A895-429CB2238AC3@nostrum.com> <e6724bd0-1ea0-3014-8836-60dc454c2982@ericsson.com> <CAB7PXwTUXUa1Euar+hXY4EzOqZ0_U-eru=e1ApjTy4a2FCBYJg@mail.gmail.com> <CAB7PXwRSFXcB5zGdNP_zqyKWUJqZAK+bKsxeyWhK6eeogqJ8dw@mail.gmail.com>
X-Mailer: Apple Mail (2.3445.102.3)
Archived-At: <https://mailarchive.ietf.org/arch/msg/sipbrandy/yMiUY1LbAZxjY5k--z_4vvBvv04>
Subject: Re: [Sipbrandy] AD Evaluation of draft-ietf-sipbrandy-osrtp-07
X-BeenThere: sipbrandy@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: SIPBRANDY working group discussion list <sipbrandy.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sipbrandy>, <mailto:sipbrandy-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sipbrandy/>
List-Post: <mailto:sipbrandy@ietf.org>
List-Help: <mailto:sipbrandy-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sipbrandy>, <mailto:sipbrandy-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 26 Mar 2019 16:10:22 -0000

(+Alexey, who will take over SIPBRANDY when I step down as AD)

Hi,

Thanks for the response. This does not quite address my main substantive comment. It does address everything else :-)

Please see comment in line.

Thanks!

Ben.

> On Mar 26, 2019, at 11:58 AM, Andy Hutton <andyhutton.ietf@gmail.com> wrote:
> 
> I submitted an update in response to Ben's comments -
> https://tools.ietf.org/html/draft-ietf-sipbrandy-osrtp-08
> 
> With regard to Ben's comment on the relaxing of the authentication
> requirement then this is consistent with the Opportunistic Security
> RFC 7435 and I added a reference to this as clarification.

If I recall correctly, RFC 7435 does not discuss scenarios with separate signaling and media channels, and how OS applies to each channel. I was looking more for something about the impacts of this “relaxation” specific to these sorts of scenarios with dtls-srtp and sdes, and resulting assurances.

For example, dtls-srtp with no authentication does not give you assurances about who you are talking to, but it still allows encryption. SDES without encryption lets an eavesdropper potentially learn the encryption keys, etc. SDES with transport level protection  (e.g. SIPS) protects from off-path eavesdroppers, but allows proxies and b2bua’s in the signaling path to learn the encryption keys.


> 
> Hopefully we can get this to RFC status now.
> 
> Regards
> Andy
> 
> On Mon, 25 Mar 2019 at 22:26, Andy Hutton <andyhutton.ietf@gmail.com> wrote:
>> 
>> Sorry about the delay.
>> 
>> See below.
>> 
>> I will update the draft.
>> 
>> Andy
>> 
>> On Fri, 15 Feb 2019 at 08:46, Gonzalo Camarillo
>> <gonzalo.camarillo@ericsson.com> wrote:
>>> 
>>> Thanks for the quick review, Ben!
>>> 
>>> Authors, please address Ben's comments below.
>>> 
>>> Cheers,
>>> 
>>> Gonzalo
>>> 
>>> On 14-Feb-19 22:46, Ben Campbell wrote:
>>>> Hi,
>>>> 
>>>> This is my AD Evaluation of draft-ietf-sipbrandy-osrtp-07.
>>>> 
>>>> Thank you for a readable and easy to understand document.There is one comment I would like to resolve prior to IETF LC. The others can be resolved along with any last call feedback.
>>>> 
>>>> *** Please resolve prior to IETF LC ***
>>>> 
>>>> §4: The relaxation of authentication requirements for DTLS-SRTP and SDES could use some elaboration on why this acceptable. I _think_ the answer is that, since OSRTP doesn’t guaranty authentication, there’s no need for such a guaranty from the signaling channel. Is that correct?
>>>> 
>>>> OTOH, §1 says "third mode for security between "cleartext” and "comprehensive protection" that allows encryption and authentication to be used if supported…”. That suggests that that authentication is sometimes provided. Is there a distinction between the authenticated case and unauthenticated case that should be mentioned somewhere? (For example, is there a need to indicate the distinction to the user?)
>>>> 
>> 
>> $1 should I think say "allows encryption and authenticated media" but
>> I cannot remember why we said the signalling authentication
>> requirements are relaxed this has been in the draft from day 1 and I
>> guess it is consistent with the best effort approach.
>> 
>> Anyone else want to comment?
>> 
>> 
>>>> *** Other Substantive Comments ***
>>>> 
>>>> §2: Please use the new boilerplate from RFC 8174.
>> 
>> Will do.
>> 
>>>> 
>>>> §3.1: Please clarify that that the offer can contain more than one key management attribute. This is mentioned in §3.1, but not actually in the section on generating the offer.
>> 
>> Will do.
>> 
>>>> 
>>>> *** Editorial Comments ***
>>>> 
>>>> §3: "As discussed in [RFC7435], this is
>>>> the "comprehensive protection" for media mode.”
>>>> s/this/that
>> 
>> Thanks
>> 
>>>> 
>>>> §3.4: "meaning that the decision to
>>>> create an OSRTP type offer or something else should not be influenced”
>>>> That’s referring to the decision to create a _new_ offer, right? Not the original offer?
>> 
>> Correct.
>> 
>>>> 
>>>> 
>>>> 
>>>> _______________________________________________
>>>> Sipbrandy mailing list
>>>> Sipbrandy@ietf.org
>>>> https://www.ietf.org/mailman/listinfo/sipbrandy
>>>> 
>>> _______________________________________________
>>> Sipbrandy mailing list
>>> Sipbrandy@ietf.org
>>> https://www.ietf.org/mailman/listinfo/sipbrandy