IETF Logo Mail Archive

Re: [bfcpbis] AD Review of draft-ietf-bfcpbis-rfc4583bis-24: role issue - The pull request

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 19 September 2018 18:06 UTC

Return-Path: <christer.holmberg@ericsson.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 8473D130E6A for <bfcpbis@ietfa.amsl.com>; Wed, 19 Sep 2018 11:06:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.309
X-Spam-Level:
X-Spam-Status: No, score=-4.309 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_MED=-2.3, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 EgDLUR4yVJYh for <bfcpbis@ietfa.amsl.com>; Wed, 19 Sep 2018 11:06:41 -0700 (PDT)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 14D5A130E0A for <bfcpbis@ietf.org>; Wed, 19 Sep 2018 11:06:40 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; d=ericsson.com; s=mailgw201801; c=relaxed/simple; q=dns/txt; i=@ericsson.com; t=1537380399; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:CC:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=5Ag405jr9d4fPkGoGPhzo/vRsSy1GwuMf6uAXGV4Ap0=; b=QVU6Zu3sO9F/F2YLnOP7DCECyHxce/5nJ4hPCagtvFUllrfmdQGQ9mu9miKGVCr8 AWelNRb3VJn9/xpnPsBf/HPVusj/p3C+nM3T6RxlOHl6522oxFblwAEkerAlsvST 8gSAOHk4J99yJI9RjwSUv4p/FoEt0puWyFbNtrrVDgQ=;
X-AuditID: c1b4fb2d-20bff700000055ff-04-5ba2902fa94f
Received: from ESESBMB505.ericsson.se (Unknown_Domain [153.88.183.118]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id D2.62.22015.F2092AB5; Wed, 19 Sep 2018 20:06:39 +0200 (CEST)
Received: from ESESBMB503.ericsson.se (153.88.183.170) by ESESBMB505.ericsson.se (153.88.183.172) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3; Wed, 19 Sep 2018 20:06:38 +0200
Received: from ESESBMB503.ericsson.se ([153.88.183.186]) by ESESBMB503.ericsson.se ([153.88.183.186]) with mapi id 15.01.1466.003; Wed, 19 Sep 2018 20:06:38 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Alan Ford <alan.ford@gmail.com>
CC: "bfcpbis@ietf.org" <bfcpbis@ietf.org>
Thread-Topic: [bfcpbis] AD Review of draft-ietf-bfcpbis-rfc4583bis-24: role issue - The pull request
Thread-Index: AdRLeDAXQdw2ACGbThquOLmME3BJeQErzWQAAAaj0IA=
Date: Wed, 19 Sep 2018 18:06:38 +0000
Message-ID: <e54f48b925af41e28cb4f290af9b64ae@ericsson.com>
References: <24eb743e07ff4af087ab36f1c4fe050e@ericsson.com> <96BA408A-8034-4558-BFEA-2FD2A2FE506C@gmail.com>
In-Reply-To: <96BA408A-8034-4558-BFEA-2FD2A2FE506C@gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.153]
Content-Type: multipart/alternative; boundary="_000_e54f48b925af41e28cb4f290af9b64aeericssoncom_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrAIsWRmVeSWpSXmKPExsUyM2J7ma7+hEXRBpsn8VisPLeC2eLfuqNM DkweO2fdZfdYsuQnUwBTFJdNSmpOZllqkb5dAlfG4ePLGQt+tTBWzFh6gbGBcUMjYxcjJ4eE gInE721HgGwuDiGBo4wSr37sYYNwvjFKPJzbwgzhLGOUOLRpMlAZBwebgIVE9z9tEFNEQFli +SxWkEHMApoSVw/vAhsqLJAq8e3tBbC4iECaxI/1bWwQ5VYSNz65gYRZBFQlTl7sZwexeQWs JS5ePAhmCwnkSbx6vAhsDKeArUTLnv1gYxgFxCS+n1rDBLFKXOLWk/lMEPcLSCzZc54ZwhaV ePn4HyuErSSx99h1Foj6ZInjqy6yQOwSlDg58wnLBEbRWUhGzUJSNgtJ2Sygq0E+W79LH6JE UWJK90N2CFtDonXOXHZk8QWM7KsYRYtTi4tz042M9VKLMpOLi/Pz9PJSSzYxAuPt4JbfujsY V792PMQowMGoxMP7omFRtBBrYllxZe4hRgkOZiUR3pWlQCHelMTKqtSi/Pii0pzU4kOM0hws SuK8eqv2RAkJpCeWpGanphakFsFkmTg4pRoYjY2+N76ee3VBXU/OFs+yovjHTHMCS0qeLeZW FXuhlaLTKfLob8Fn15L/+163Z5l+c809Yql7nbOQXTTbb+7sX0edJAwWOjPrneOcGnqVLfXq TvP3m1RO387vD5pR+vZABeebkxsEa9MX5kwU1t9q6P1G9r/XXo3fChkL+FfuqJ3Nct5h7+NM JZbijERDLeai4kQA+tE2/rMCAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/bfcpbis/-rnFsgNGhI_EYzeATqNRYRaE5LI>
Subject: Re: [bfcpbis] AD Review of draft-ietf-bfcpbis-rfc4583bis-24: role issue - The pull request
X-BeenThere: bfcpbis@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 19 Sep 2018 18:06:44 -0000

Hi Alan,

A few points regarding your suggested text:


1)      There is no BFCPv2. We need to talk about “An implementation compliant to this specification”, or something like that.

2)      I think we shall say “MUST accept "c-s" as equivalent to "c-only s-only", because that is consistent and testable.

3)      Instead of saying “was not implemented as specified”, I would say “was underspecified, and implemented in different ways, e.g., some implementations interpreted ”c-s” meaning that endpoint is willing to act as either client or server (equivalent to “c-only s-only”).”.

Regards,

Christer

From: Alan Ford [mailto:alan.ford@gmail.com]
Sent: 19 September 2018 19:45
To: Christer Holmberg <christer.holmberg@ericsson.com>
Cc: bfcpbis@ietf.org
Subject: Re: [bfcpbis] AD Review of draft-ietf-bfcpbis-rfc4583bis-24: role issue - The pull request

Hi Christer, all,

I’ve just added some review comments to the PR. One sentence still needs to be removed, and I think we need to be stronger with the clarification of the deprecation, I propose a paragraph along the lines of:

In RFC4583, there was a third option specified, "c-s", which meant that an endpoint was willing to act as floor control client and floor control server. This feature was not implemented as specified, and the implementations which did signal "c-s" used it to mean that the endpoint is willing to act as floor control client or floor control server. An implementation of BFCPv2 MUST NOT send "c-s" as a floorctl attribute value, but I
Regards,
Alan


On 13 Sep 2018, at 16:43, Christer Holmberg <christer.holmberg@ericsson.com<mailto:christer.holmberg@ericsson.com>> wrote:

Hi,

I have created a pull request for the role issue.

https://github.com/cdh4u/draft-bfcp-4583bis/pull/10

The pull request also updates references but does not address Adam's other issues.

Regards,

Christer

Ps. Note that I do not intend to submit a pull request for 4582bis. If changes are needed, I assume one of the authors will handle that.

_______________________________________________
bfcpbis mailing list
bfcpbis@ietf.org<mailto:bfcpbis@ietf.org>
https://www.ietf.org/mailman/listinfo/bfcpbis

v2.23.0 | Report a Bug | By Email