Gen-ART review of draft-melnikov-sieve-imapext-metadata-08
"Spencer Dawkins" <spencer@wonderhamster.org> Tue, 23 December 2008 18:05 UTC
Return-Path: <ietf-bounces@ietf.org>
X-Original-To: ietf-archive@megatron.ietf.org
Delivered-To: ietfarch-ietf-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id BF47E3A6B2F; Tue, 23 Dec 2008 10:05:44 -0800 (PST)
X-Original-To: ietf@core3.amsl.com
Delivered-To: ietf@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 092893A6B2A; Tue, 23 Dec 2008 10:05:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.598
X-Spam-Level:
X-Spam-Status: No, score=-2.598 tagged_above=-999 required=5 tests=[AWL=-0.000, BAYES_00=-2.599, STOX_REPLY_TYPE=0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id BYnFu5aD65Di; Tue, 23 Dec 2008 10:05:42 -0800 (PST)
Received: from mout.perfora.net (mout.perfora.net [74.208.4.194]) by core3.amsl.com (Postfix) with ESMTP id D2B8B3A6452; Tue, 23 Dec 2008 10:05:42 -0800 (PST)
Received: from S73602b (cpe-72-190-78-151.tx.res.rr.com [72.190.78.151]) by mrelay.perfora.net (node=mrus1) with ESMTP (Nemesis) id 0MKpCa-1LFBdF0Dcy-0007bN; Tue, 23 Dec 2008 13:05:29 -0500
Message-ID: <7CA30E7007064EB6B8929039C4604804@china.huawei.com>
From: Spencer Dawkins <spencer@wonderhamster.org>
To: Alexey Melnikov <alexey.melnikov@isode.com>
Subject: Gen-ART review of draft-melnikov-sieve-imapext-metadata-08
Date: Tue, 23 Dec 2008 12:05:06 -0600
MIME-Version: 1.0
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2900.5512
X-MIMEOLE: Produced By Microsoft MimeOLE V6.00.2900.5579
X-Provags-ID: V01U2FsdGVkX19mIeLI7S0NOcg5xZqMUlBuOKD9OPYpaD3xxwI +zJOnt321oR2fDe/EsizgN+91BpuTo7CF/dJ3MUINbvnYdO3oN YWnziyGoVU6jzyaWIEnwVQcUwiZ185AHGyAAXespfs=
Cc: General Area Review Team <gen-art@ietf.org>, Lisa Dusseault <lisa@osafoundation.org>, ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Sender: ietf-bounces@ietf.org
Errors-To: ietf-bounces@ietf.org
I have been selected as the General Area Review Team (Gen-ART) reviewer for this draft (for background on Gen-ART, please see http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). Please resolve these comments along with any other Last Call comments you may receive. Document: draft-melnikov-sieve-imapext-metadata-08 Reviewer: Spencer Dawkins Review Date: 2008-12-23 IETF LC End Date: 2009-01-14 IESG Telechat date: (not known) Summary: Almost ready for publication as a Proposed Standard. Please see notes below. Comments: 3. mailbox and mboxmetadata extensions 3.1. Test mailboxexists Note that a successful "mailboxexists" test for a mailbox doesn't necessarily mean that a "fileinto" action on this mailbox would succeed. For example the "fileinto" action might put user over quota. The "mailboxexists" only verifies existence of the mailbox and whether the user in whose context the Sieve script runs has permissions to execute fileinto on it. Spencer (nit): you've been putting "fileinto" in quotes, up to this point - suggest that this be consistent. The capability string for use with the require command is "mailbox". Example: The following example assumes that the Sieve engine also supports "reject" [REJECT] and "fileinto" [SIEVE]. However these extensions are not required in order to implement the "mailbox" extension. require ["fileinto", "reject", "mailbox"]; if mailboxexists "Partners" { fileinto "Partners"; } else { reject "This message was not accepted by the Mailstore"; } 5. Security Considerations Extensions defined in this document deliberately don't provide a way to modify annotations. Spencer: The next two paragraphs punt to "same as sieve script" - could you provide a specific reference for the reader here? Just the reference would be fine, nothing else needed. A failure to retrieve data due to the server storing the annotations being down or otherwise inaccessible may alter the result of Sieve processing. So implementations SHOULD treat a temporary failure to retrieve annotations in the same manner as a temporary failure to retrieve a Sieve script. Protocols/APIs used to retrieve annotations MUST provide the same level of confidentiality as protocols/APIs used to retrieve Sieve scripts. 6. IANA Considerations IANA is requested to add the following registrations to the list of Sieve extensions: To: iana@iana.org Subject: Registration of new Sieve extension Capability name: mailbox Description: adds test for checking for mailbox existence and a new optional argument to fileinto for creating a mailbox before attempting mail delivery. RFC number: this RFC Spencer: you probably want to add notes to IANA and the RFC Editor to replace "this RFC" with the RFC number when it is assigned (just make it easier for them). Contact address: The Sieve discussion list <ietf-mta-filters@imc.org> Capability name: mboxmetadata Description: adds tests for checking for mailbox metadata item existence and for retrieving of a mailbox metadata value. RFC number: this RFC Contact address: The Sieve discussion list <ietf-mta-filters@imc.org> Capability name: servermetadata Description: adds tests for checking for server metadata item existence and for retrieving of a server metadata value. RFC number: this RFC Contact address: The Sieve discussion list <ietf-mta-filters@imc.org> _______________________________________________ Ietf mailing list Ietf@ietf.org https://www.ietf.org/mailman/listinfo/ietf
- Gen-ART review of draft-melnikov-sieve-imapext-me… Spencer Dawkins
- Re: Gen-ART review of draft-melnikov-sieve-imapex… Alexey Melnikov
- Re: Gen-ART review of draft-melnikov-sieve-imapex… Spencer Dawkins
- Re: Gen-ART review of draft-melnikov-sieve-imapex… Alexey Melnikov
- Re: Gen-ART review of draft-melnikov-sieve-imapex… Spencer Dawkins