Re: [sieve] Duplicate test

NED+mta-filters@mauve.mrochek.com Fri, 18 January 2013 19:41 UTC

Return-Path: <NED+mta-filters@mauve.mrochek.com>
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 D06C021F8790 for <sieve@ietfa.amsl.com>; Fri, 18 Jan 2013 11:41:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599]
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 1FzhpbSLFEPX for <sieve@ietfa.amsl.com>; Fri, 18 Jan 2013 11:41:49 -0800 (PST)
Received: from mauve.mrochek.com (mauve.mrochek.com [66.59.230.40]) by ietfa.amsl.com (Postfix) with ESMTP id F1F9821F8795 for <sieve@ietf.org>; Fri, 18 Jan 2013 11:41:48 -0800 (PST)
Received: from dkim-sign.mauve.mrochek.com by mauve.mrochek.com (PMDF V6.1-1 #35243) id <01OP4U4Z4K34005L9C@mauve.mrochek.com> for sieve@ietf.org; Fri, 18 Jan 2013 11:36:47 -0800 (PST)
Received: from mauve.mrochek.com by mauve.mrochek.com (PMDF V6.1-1 #35243) id <01OOJ8U2Q2U800008S@mauve.mrochek.com> (original mail from NED@mauve.mrochek.com) for sieve@ietf.org; Fri, 18 Jan 2013 11:36:45 -0800 (PST)
From: NED+mta-filters@mauve.mrochek.com
Message-id: <01OP4U4XTT0U00008S@mauve.mrochek.com>
Date: Fri, 18 Jan 2013 11:22:30 -0800
In-reply-to: "Your message dated Wed, 16 Jan 2013 22:36:19 +0000" <50F72B63.3080004@isode.com>
MIME-version: 1.0
Content-type: TEXT/PLAIN; Format="flowed"
References: <50F3F8C7.9030804@rename-it.nl> <50F72B63.3080004@isode.com>
To: Alexey Melnikov <alexey.melnikov@isode.com>
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: Fri, 18 Jan 2013 19:41:49 -0000

> 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.

				Ned