Re: [clue] WGLC for draft-ietf-clue-protocol-10

Christian Groves <Christian.Groves@nteczone.com> Tue, 24 January 2017 05:08 UTC

Return-Path: <Christian.Groves@nteczone.com>
X-Original-To: clue@ietfa.amsl.com
Delivered-To: clue@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 77DA3129568 for <clue@ietfa.amsl.com>; Mon, 23 Jan 2017 21:08:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.79
X-Spam-Level:
X-Spam-Status: No, score=-1.79 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, T_DKIM_INVALID=0.01] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=neutral reason="invalid (public key: not available)" header.d=nteczone.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 UN0MrZjuqBHJ for <clue@ietfa.amsl.com>; Mon, 23 Jan 2017 21:08:51 -0800 (PST)
Received: from msh03.myshophosting.com (msh03.myshophosting.com [101.0.109.158]) (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 A6025126CD8 for <clue@ietf.org>; Mon, 23 Jan 2017 21:08:51 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=nteczone.com; s=default; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:Cc:References:To:Subject:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=udWm9JUKRDC9dx6uy+qwlVOmy7wkG8HNmffxu7vnRYg=; b=FS5GarNcvZQsSMjzJEq5Nvt9Vk 0bFayndDoGUU/vtHYzXs+mN53bTGvVVgTZlPU2yd4FdE7O29hSTpRydem9R7zPJVROQZQbD4uhr0x Md7fEbd0ag9dprCOnhqrVqtiNDYMlCeFnT0a8oKwR2xIFX1fXDdfoRVoUYd2jElx00/JTShAytPF6 yEnuPB4AMHgcBIWkmXMPqsfjb2fXh4B8chQqV+f4uuwwWePz+wiwPpwhxQxVO6ynnl1hYg6zcI8wn p08TLXZs2Hxs69CJmk8t7JsdF2X7HMUtz2QzZdv57Lo/AGyImVgosxCHJfFHGVP1vXsdfCpomK5gY q04jty0w==;
Received: from ppp118-209-52-63.lns20.mel4.internode.on.net ([118.209.52.63]:49778 helo=[192.168.1.22]) by msh03.myshophosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.87) (envelope-from <Christian.Groves@nteczone.com>) id 1cVtLc-003bLO-Ey; Tue, 24 Jan 2017 16:08:48 +1100
To: Simon Pietro Romano <spromano@unina.it>
References: <ac44e23d-061b-5d1b-b6e5-24e8f5ef0ffc@alum.mit.edu> <075716a0-ab1d-f943-50d0-a65fd339f165@nteczone.com> <4B2480BA-75CA-4E73-A3D4-ABA3058EE6AD@unina.it> <e220de50-db77-e021-c824-1d246f2eb2dd@nteczone.com> <8A070EF8-BEB7-4CA8-86C1-E10A25C91F04@unina.it>
From: Christian Groves <Christian.Groves@nteczone.com>
Message-ID: <5f5ddf96-41a9-9c55-c692-077791a04ec7@nteczone.com>
Date: Tue, 24 Jan 2017 16:08:45 +1100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0
MIME-Version: 1.0
In-Reply-To: <8A070EF8-BEB7-4CA8-86C1-E10A25C91F04@unina.it>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - msh03.myshophosting.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - nteczone.com
X-Get-Message-Sender-Via: msh03.myshophosting.com: authenticated_id: christian.groves@nteczone.com
X-Authenticated-Sender: msh03.myshophosting.com: christian.groves@nteczone.com
X-Source:
X-Source-Args:
X-Source-Dir:
Archived-At: <https://mailarchive.ietf.org/arch/msg/clue/5Q6chcALSt_gVmHR4WWXzL2AIsY>
Cc: clue@ietf.org
Subject: Re: [clue] WGLC for draft-ietf-clue-protocol-10
X-BeenThere: clue@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: CLUE - ControLling mUltiple streams for TElepresence <clue.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/clue>, <mailto:clue-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/clue/>
List-Post: <mailto:clue@ietf.org>
List-Help: <mailto:clue-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/clue>, <mailto:clue-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Jan 2017 05:08:53 -0000

Hello Simon and Roberta,

Thanks for addressing the comments. Please see below.

Regards, Christian

..snip
>
>>> Cl.5.7: It says future protocol version can introduce error codes? 
>>> How about options? It seems like an option could introduce a 
>>> specific error code.
>>> Based on the discussion in section 8, a new response code might 
>>> indeed be added through the “extension” mechanism. Though, the 
>>> “cleanest” way for achieving such a task in a formal way is by adding
>>> the new response code to the list of codes published in the 
>>> dedicated IANA registry (see section 12.4.2 fo the draft). In the 
>>> mentioned section, we refer to future versions of the standard.
>> [CNG] By saying that error codes can be designed in future versions 
>> seems to preclude them being defined by extensions. (BTW 5.7 uses 
>> "designed", I think "defined" would be better).
>
> We rephrased the sentence as follows:
>
> "Further response codes can be either defined in future versions of the
> protocol  or defined
> by leveraging the extension mechanism. In any case, such new response
> codes MUST NOT overwrite the ones here defined and they MUST
> respect the semantics of the first code digit.”
>
> Does this sound OK to you?
[CNG] I agree with Paul in that any new response code will need to be 
registered by IANA. However I don't see the problem with having two 
mechanisms to define codes within the protocol. It should be possible to 
add an error code to CLUE without having to bump the version. I think if 
you delete "(by adding them to the related IANA registry)," and add a 
sentence along the lines of:
"In both cases the new response code MUST be registered with IANA".

>
>> I agree one could simply update the IANA registry with the new code 
>> but I was thinking more of how an end point would know that it would 
>> be used. That's where registering an extension would make sense. Of 
>> course if you made a new version you wouldn't need an extension.
>
> OK. See above.
>
>> ..snip
>>> Cl.6 para 5: "Otherwise <if> ("channel error")…"
>>> Should we add an "if" between “Otherwise” and the parenthesis? 
>>> Please advise on that.
>> [CNG] If doesn't need to be in parentheses just "Otherwise if …"
>
> You’re the English master! We modified accordingly:
>
> "Otherwise if ("channel error"), it moves back to the IDLE state.”
[CNG] You can delete the ( and ), i.e. just "Otherwise if "channel 
error", it... "
>