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

Jayantheesh S B <j.sb@sea.samsung.com> Tue, 15 December 2015 17:26 UTC

Return-Path: <j.sb@sea.samsung.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 B85DB1A90F7; Tue, 15 Dec 2015 09:26:38 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.2
X-Spam-Level:
X-Spam-Status: No, score=-1.2 tagged_above=-999 required=5 tests=[BAYES_05=-0.5, RCVD_IN_DNSWL_LOW=-0.7] 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 QaMfYcM-51ic; Tue, 15 Dec 2015 09:26:36 -0800 (PST)
Received: from wguard02.sdsamerica.net (bware2.sdsamerica.net [206.67.236.192]) by ietfa.amsl.com (Postfix) with ESMTP id 931B71A90F4; Tue, 15 Dec 2015 09:26:36 -0800 (PST)
From: Jayantheesh S B <j.sb@sea.samsung.com>
To: Adrien de Croy <adrien@qbik.com>, Barry Leiba <barryleiba@computer.org>, Narendra Bisht <ns.bisht@sea.samsung.com>
Thread-Topic: [imapext] AD review of draft-ietf-imapapnd-appendlimit-extension-06
Thread-Index: AQHRMRx9gijbZuD2bkC3YmuL4+ER+J7DNZQAgAG44QCAAQU5gIAGZUrA
Date: Tue, 15 Dec 2015 17:26:32 +0000
Message-ID: <271b24f638804328a92121134b8d2b10@SEAMBX01.sea.samsung.com>
References: <CALaySJK=5nkmF2K0Vt7mgg2honoX9iYS4yhgu+giDjKyDoR0GQ@mail.gmail.com> <eme8fe9e99-d1fd-4ad0-88f3-65aad425c998@bodybag>
In-Reply-To: <eme8fe9e99-d1fd-4ad0-88f3-65aad425c998@bodybag>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-transport-fromentityheader: Hosted
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Received-SPF: none
Archived-At: <http://mailarchive.ietf.org/arch/msg/imapext/DNYapv0dAzq7EyqKAfmq-N5QRc8>
Cc: "draft-ietf-imapapnd-appendlimit-extension@ietf.org" <draft-ietf-imapapnd-appendlimit-extension@ietf.org>, "imapext@ietf.org" <imapext@ietf.org>
Subject: Re: [imapext] AD review of 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: Tue, 15 Dec 2015 17:26:38 -0000

Hi All,

Please find our response updated. 

Regards,
Jay
-----Original Message-----
From: imapext [mailto:imapext-bounces@ietf.org] On Behalf Of Adrien de Croy
Sent: Friday, December 11, 2015 5:45 AM
To: Barry Leiba; Narendra Bisht
Cc: draft-ietf-imapapnd-appendlimit-extension@ietf.org; imapext@ietf.org
Subject: Re: [imapext] AD review of draft-ietf-imapapnd-appendlimit-extension-06



------ Original Message ------
From: "Barry Leiba" <barryleiba@computer.org>

>>  -- Section 4 --
>>
>>  "Client can avoid use of LITERAL+ [RFC2088], when maximum upload size
>>   supported by the IMAP server is unknown."
>>
>>  What?
>>  Don't you mean "The client SHOULD avoid"?  I'd even use this as an 
>>opportunity
>>  to make it firmer, and say "The client MUST avoid".  No?
>>  If not, why not?
>>
>>   [Naren] We will change it to a MUST
>
>Hold off on this, because there's still discussion based on Adrian's 
>message in another thread... which I'll bring back here:
>
>On Tue, Dec 8, 2015 at 4:31 PM, Adrien de Croy <adrien@qbik.com> wrote:
>>
>>  The proposal that a client MUST avoid LITERAL+/NSLs presumes there 
>>is a
>>  limit when in fact there may actually not be one.  Of course there 
>>is always
>>  a finite limit, but there may be no policy limit.  In fact we don't 
>>plan to
>>  implement the limit as we've never had a request for it and don't 
>>see a need
>>  to deny authenticated users from appending a mail (and see some 
>>dangers in
>>  that).
>>
>>  I think MAY works in that it proposes a strategy, and doesn't 
>>confuse issues
>>  with servers that already implement LITERAL+ but not a limit.  
>>Otherwise you
>>  may be placing a new requirement on old software to police the new 
>>MUST, or
>>  implementing the limit places addition requirements to alter 
>>behaviour of
>>  LITERAL+ support to enforce this which IMO over-complicates it.
>
>But the point of the use of LITERAL+ with APPEND isn't just about this 
>spec and overall limits -- it's about whether we should use LITERAL+ 
>with APPEND *at all*.  There are other reasons that any particular 
>APPEND might fail, and one point of using literals (and *never* 
>allowing quoted strings, for example) is exactly to give the server a 
>chance to say "NO" to the APPEND *before* the message data is shipped 
>over.  Using LITERAL+ for APPEND data violates that.
OK fair point, the extra round trip for APPEND only isn't that big a deal.

Do we need to ad an addenda to LITERAL+ then?

>
>It was always the intent of LITERAL+ that it be used as a way to 
>eliminate the extra round trip on short strings, where the OK from the 
>server isn't necessary -- such as for username and password at login, 
>or for mailbox names in various places (including the mailbox name in 
>an APPEND command.
>
>My point here is that we now have an opportunity to stress this: that 
>it's not a generally good idea to use LITERAL+ for the message data in 
>an APPEND command, because it doesn't give the server the opportunity 
>to say, "OK, yes, go ahead and send me the message."
>
>I'm absolutely willing to accept that "MUST NOT" use LITERAL+ for that 
>is too strong.  But I'm still going to hold out for "SHOULD NOT", and 
>would like to continue the discussion of why you disapprove.

Happy with the SHOULD NOT.


[Jay] We will update the draft with SHOULD  NOT

Adrien

_______________________________________________
imapext mailing list
imapext@ietf.org
https://www.ietf.org/mailman/listinfo/imapext