ietf-mta-filters
[Top] [All Lists]

Re: [sieve] I-D Action:draft-ietf-sieve-external-lists-08.txt

2011-05-19 12:58:33
Yes, but strictly speaking '"addrbook" URI' is not correct.
...
That is the only place I've noticed.

Changed in my working copy.

Cyrus, Ned -- can you review soon?  Thanks.

I just did a complete review of -09, and also updated our implementation to
match. The one thing I wanted to point out is that section 2.6 is going to end
up in the IANA registry in it's entirety, including the text about the example
parameter values. I think this is OK, but want to be sure nobody else thinks it
might confuse people into thinking all these names actually need to be
implemented.

The only other thing that occurred to me is that we could add a variant
of example 2 showing how to use string :list if currentdate doesn't
support :list directly. Something like:

       require ["extlists", "date", "enotify", "variables"];
       if currentdate :matches "date" *" {
         set "cd" "${0}";
         if string :list "${cd}" "date"
            "tag:example.com,2011-01-01:localHolidays" {
            notify "xmpp:romeo(_at_)im(_dot_)example(_dot_)com";
         }
       }

I certainly don't insist on this addition but I thought it might be helpful.

                                Ned
_______________________________________________
sieve mailing list
sieve(_at_)ietf(_dot_)org
https://www.ietf.org/mailman/listinfo/sieve