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

[email protected]
Newsgroups gmane.ietf.mta-filters
Message-ID <[email protected]>
> > 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:[email protected]";
         }
       }

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

				Ned
_______________________________________________
sieve mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/sieve
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.