Re: I-D ACTION:draft-ietf-sieve-variables-04.txt

Ned Freed <ned.freed@mrochek.com> Wed, 20 July 2005 18:25 UTC

Received: from above.proper.com (localhost.vpnc.org [127.0.0.1]) by above.proper.com (8.12.11/8.12.9) with ESMTP id j6KIPKlp002987; Wed, 20 Jul 2005 11:25:20 -0700 (PDT) (envelope-from owner-ietf-mta-filters@mail.imc.org)
Received: (from majordom@localhost) by above.proper.com (8.12.11/8.12.9/Submit) id j6KIPKD4002986; Wed, 20 Jul 2005 11:25:20 -0700 (PDT)
X-Authentication-Warning: above.proper.com: majordom set sender to owner-ietf-mta-filters@mail.imc.org using -f
Received: from mauve.mrochek.com (mauve.mrochek.com [209.55.107.55]) by above.proper.com (8.12.11/8.12.9) with ESMTP id j6KIPJ84002979 for <ietf-mta-filters@imc.org>; Wed, 20 Jul 2005 11:25:19 -0700 (PDT) (envelope-from ned.freed@mrochek.com)
Received: from mauve.mrochek.com by mauve.mrochek.com (PMDF V6.1-1 #35243) id <01LQT93SRW80000092@mauve.mrochek.com> for ietf-mta-filters@imc.org; Wed, 20 Jul 2005 11:25:15 -0700 (PDT)
Cc: ietf-mta-filters@imc.org
To: Kjetil Torgrim Homme <kjetilho@ifi.uio.no>
Message-id: <01LQUHIQ7UJA000092@mauve.mrochek.com>
Date: Wed, 20 Jul 2005 10:51:42 -0700
From: Ned Freed <ned.freed@mrochek.com>
Subject: Re: I-D ACTION:draft-ietf-sieve-variables-04.txt
In-reply-to: "Your message dated Sat, 16 Jul 2005 16:09:36 +0200" <1121522976.8017.6.camel@chico.njus.no>
MIME-version: 1.0
Content-type: TEXT/PLAIN
References: <E1DtRVp-0002k4-CW@newodin.ietf.org> <1121522976.8017.6.camel@chico.njus.no>
Sender: owner-ietf-mta-filters@mail.imc.org
Precedence: bulk
List-Archive: <http://www.imc.org/ietf-mta-filters/mail-archive/>
List-ID: <ietf-mta-filters.imc.org>
List-Unsubscribe: <mailto:ietf-mta-filters-request@imc.org?body=unsubscribe>


> On Fri, 2005-07-15 at 10:50 -0400, Internet-Drafts@ietf.org wrote:
> > A URL for this Internet-Draft is:
> > http://www.ietf.org/internet-drafts/draft-ietf-sieve-variables-04.txt

> the changes are:

> * added :quotewildcard and :quoteregex mofiefiers to SET.

Basic idea looks good, however, a couple of typos have crept in. First, the
syntax at the begining of 4.1 fails to list the two new modifiers. Second, the
plain text draft now says:

4.1.2.1.  Modifier ":quotewildcard"

   Every character with special meaning for :matches ("*", "?" and " is
   prefixed with "

4.1.2.2.  Modifier ":quoteregex"

   Every character with special meaning for :regex (".", "*", "?" etc.)
   is prefixed with "

I assume you meant '"\"' and not '"'. Looks like you need to apply some quoting
to the draft source...

The lack of a full list of regex specials is a bit awkward here, but I
guess I can live with it.

Finally, the section ends with "Using two or more modifiers of the same
precedence is a syntax error.". This isn't quite right: These aren't syntax
errors. How about saying "It is an error to use two or more modifiers of the
same precedence in a single SET operation" or something similar.

> * added this paragraph to security considerations:

>         Since values stored by SET exceeding implementation limits are
>         silently truncated, it's not appropriate to store large
>         structures with security implications in variables.

> I did not remove the text about interaction with regex yet.  I feel the
> text belongs in the variables draft, so if it is procedurally possible,
> I prefer it stays there rather than move into the regex draft.

I also would like to keep the text here if possible.

				Ned