Re: [sieve] Duplicate test

Stephan Bosch <stephan@rename-it.nl> Sun, 20 January 2013 13:00 UTC

Return-Path: <stephan@rename-it.nl>
X-Original-To: sieve@ietfa.amsl.com
Delivered-To: sieve@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2FC1221F84E9 for <sieve@ietfa.amsl.com>; Sun, 20 Jan 2013 05:00:57 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.504
X-Spam-Level:
X-Spam-Status: No, score=-0.504 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HELO_EQ_NL=0.55, HOST_EQ_NL=1.545]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id n+-Czm0bj350 for <sieve@ietfa.amsl.com>; Sun, 20 Jan 2013 05:00:56 -0800 (PST)
Received: from drpepper.rename-it.nl (drpepper.rename-it.nl [217.119.238.16]) by ietfa.amsl.com (Postfix) with ESMTP id A936221F84E7 for <sieve@ietf.org>; Sun, 20 Jan 2013 05:00:56 -0800 (PST)
Received: from klara.student.utwente.nl ([130.89.162.218]:57548 helo=[10.168.3.2]) by drpepper.rename-it.nl with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.72) (envelope-from <stephan@rename-it.nl>) id 1TwuW5-0004rd-8m; Sun, 20 Jan 2013 14:00:54 +0100
Message-ID: <50FBEA74.90603@rename-it.nl>
Date: Sun, 20 Jan 2013 14:00:36 +0100
From: Stephan Bosch <stephan@rename-it.nl>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130107 Thunderbird/17.0.2
MIME-Version: 1.0
To: Ned Freed <ned.freed@mrochek.com>
References: <50F3F8C7.9030804@rename-it.nl> <50F72B63.3080004@isode.com> <01OP4U4XTT0U00008S@mauve.mrochek.com>
In-Reply-To: <01OP4U4XTT0U00008S@mauve.mrochek.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-RenameIT-MailScanner-SpamScore: -2.3 (--)
X-RenameIT-MailScanner-SpamCheck: No, score=-2.3 required=5.0 tests=ALL_TRUSTED, BAYES_00 autolearn=ham version=3.3.1
Cc: Sieve mailing list <sieve@ietf.org>
Subject: Re: [sieve] Duplicate test
X-BeenThere: sieve@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: SIEVE Working Group <sieve.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sieve>, <mailto:sieve-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sieve>
List-Post: <mailto:sieve@ietf.org>
List-Help: <mailto:sieve-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sieve>, <mailto:sieve-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 20 Jan 2013 13:00:57 -0000

On 1/18/2013 8:22 PM, Ned Freed wrote:
>> 2). I found that "the side effect of the test takes force at the end of
>> a successful script execution" to be odd/awkward. Maybe we can have an
>> explicit action? Or maybe leaving this as is is Ok... Need to think a
>> bit more about that.
>
> I'm not wild about this either, mostly because when you do this as two 
> steps
> there's a window where overlapping script executions can cause a 
> duplicate to
> be missed. Of course locking can be used to prevent this, but given 
> the scale
> we operate at any use of locking has to be carefully considered.
>
> That said, the problem where a script goes wonky and ends up marking 
> something
> as a duplicate is much more severe, so IMO the draft is handling this 
> the right
> way.
>
> tl;dr: Missing a duplicate is preferable to marking a message as a 
> duplicate
> incorrectly.

Exactly.

Perhaps I should mention these considerations more explicitly in the draft?

Regards,

Stephan.