Re: [imapext] AD review draft-ietf-imapapnd-appendlimit-extension-06

Alexey Melnikov <alexey.melnikov@isode.com> Thu, 17 December 2015 17:58 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: imapext@ietfa.amsl.com
Delivered-To: imapext@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0FC0C1B3007 for <imapext@ietfa.amsl.com>; Thu, 17 Dec 2015 09:58:30 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.011
X-Spam-Level:
X-Spam-Status: No, score=-2.011 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=ham
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 J0_F95exacOE for <imapext@ietfa.amsl.com>; Thu, 17 Dec 2015 09:58:28 -0800 (PST)
Received: from statler.isode.com (Statler.isode.com [62.232.206.189]) by ietfa.amsl.com (Postfix) with ESMTP id 284141B2FF6 for <imapext@ietf.org>; Thu, 17 Dec 2015 09:58:28 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1450375107; d=isode.com; s=selector; i=@isode.com; bh=SkhCFVaX5alCSIv3s1FxYWprkzh57PSPJmhIKKf2N54=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=I2rXK1pCXUZMoXSAd8TLUsvvoSzkTq/ZE4UH1OLSRITw7DZNyzUfbk20VwUv6sZzx5Ywug 9uLzsLwktSJ64pPZQ1dj+1M8W/mnDVRe1ctDx+TZqfOujjJ8uysY3Y9XarACvlVc/Bsam2 0qo4+yKxtUlimFWNk1x28TKtUcMjk4w=;
Received: from [172.20.1.215] (dhcp-215.isode.net [172.20.1.215]) by statler.isode.com (submission channel) via TCP with ESMTPSA id <VnL3wgBBx76q@statler.isode.com>; Thu, 17 Dec 2015 17:58:27 +0000
To: Stu Brandt <stuart.brandt@teamaol.com>, imapext@ietf.org
References: <emcf7f771e-a84b-4df3-b9ff-06dd5417a655@bodybag> <5A5084CC-6733-45DB-B3D5-4F73285257D0@isode.com> <6679218db47f443794b1ce28452623eb@SEAMBX07.sea.samsung.com> <5672EEDE.5060101@isode.com> <5672F2BE.8060404@teamaol.com>
From: Alexey Melnikov <alexey.melnikov@isode.com>
Message-ID: <5672F7AC.9000703@isode.com>
Date: Thu, 17 Dec 2015 17:58:04 +0000
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0
In-Reply-To: <5672F2BE.8060404@teamaol.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/imapext/LAszP5tyMpB_3iGu9qaBbOyDFV8>
Subject: Re: [imapext] AD review draft-ietf-imapapnd-appendlimit-extension-06
X-BeenThere: imapext@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Discussion of IMAP extensions <imapext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/imapext>, <mailto:imapext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/imapext/>
List-Post: <mailto:imapext@ietf.org>
List-Help: <mailto:imapext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/imapext>, <mailto:imapext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Dec 2015 17:58:30 -0000

On 17/12/2015 17:37, Stu Brandt wrote:
> Since RFC 3501 section 4.5 states...
>
>    The special form "NIL" represents the non-existence of a particular
>    data item that is represented as a string or parenthesized list, as
>    distinct from the empty string "" or the empty parenthesized list ().
>
>
> ...does the introduction of nil as an alternative to number now imply 
> that an update to this text is necessary?
We can consider it, but I don't think this is actually a big deal. 
Because the text above doesn't say that "nil" can't be used anywhere 
else ;-).
>
> - Stuart
>
> On 12/17/15 12:20 PM, Alexey Melnikov wrote:
>> Hi,
>>
>> On 17/12/2015 17:01, Narendra Bisht wrote:
>>> The data type for APPENDLIMIT is NUMBER.
>>> Will it accommodate a NIL?
>>
>> RFC 3501 defines:
>> status-att-list =  status-att SP number *(SP status-att SP number)
>>
>>
>> RFC 4466 redefines status-att-list:
>> status-att-list = status-att-val *(SP status-att-val)
>>                       ;; Redefines status-att-list from RFC 3501.
>>
>> status-att-val  = ("MESSAGES" SP number) /
>>                       ("RECENT" SP number) /
>>                       ("UIDNEXT" SP nz-number) /
>>                       ("UIDVALIDITY" SP nz-number) /
>>                       ("UNSEEN" SP number)
>>                       ;; Extensions to the STATUS responses
>>                       ;; should extend this production.
>>                       ;; Extensions should use the generic
>>                       ;; syntax defined by tagged-ext.
>>
>> So, I think you should add RFC 4466 to Normative references, then add
>> the following to Section 5 of your draft:
>>
>> appendlimit-status-att-val = "APPENDLIMIT" SP (number / nil)
>> status-att-val /= appendlimit-status-att-val
>>                              ;; status-att-val is defined in RFC 4466
>>
>>>
>>> -----Original Message-----
>>> From: imapext [mailto:imapext-bounces@ietf.org] On Behalf Of Alexey
>>> Melnikov
>>> Sent: Thursday, December 17, 2015 5:16 AM
>>> To: imapext@ietf.org
>>> Subject: Re: [imapext] AD review of
>>> draft-ietf-imapapnd-appendlimit-extension-06
>>>
>>>
>>>> On 17 Dec 2015, at 03:39, Adrien de Croy <adrien@qbik.com> wrote:
>>>>
>>>> ------ Original Message ------
>>>> From: "Barry Leiba" <barryleiba@computer.org>
>>>> To: "Bron Gondwana" <brong@fastmail.fm>
>>>> Cc: "imapext@ietf.org" <imapext@ietf.org>
>>>> Sent: 17/12/2015 9:14:31 a.m.
>>>> Subject: Re: [imapext] AD review of
>>>> draft-ietf-imapapnd-appendlimit-extension-06
>>>>
>>>>>>> Why do we think there's anything wrong with using, say, the word
>>>>>>> "NONE" (either case-insensitive or case-sensitive; I don't care)?
>>>>>>> What's wrong with this?:
>>>>>>>
>>>>>>> C: t1 STATUS INBOX (APPENDLIMIT)
>>>>>>> S: * STATUS INBOX (APPENDLIMIT NONE)
>>>>>>> S: t1 OK STATUS completed
>>>>>> If you're going to do that, why not make it (APPENDLIMIT NIL), which
>>>>>> already has exactly the meaning that we want?
>>>>> Yes, I'd forgotten about the existing token "NIL".  I like that
>>>>> approach.
>>>> +1
>>>>
>>>> Definitely not a fan of overloading the meaning of magic numbers
>>> I don't think this is a particular important point (there are always
>>> limits of some kind), but I like NIL.
>>
>> _______________________________________________
>> imapext mailing list
>> imapext@ietf.org
>> https://www.ietf.org/mailman/listinfo/imapext
>
> _______________________________________________
> imapext mailing list
> imapext@ietf.org
> https://www.ietf.org/mailman/listinfo/imapext