Hi Stephan,
Stephan Bosch wrote:
On 7/25/2011 8:18 PM, internet-drafts(_at_)ietf(_dot_)org wrote:
A New Internet-Draft is available from the on-line Internet-Drafts
directories. This draft is a work item of the Sieve Mail Filtering
Language Working Group of the IETF.
Title : Sieve Extension for converting messages before
delivery
Author(s) : Alexey Melnikov
Qian Sun
Barry Leiba
Kepeng Li
Filename : draft-ietf-sieve-convert-02.txt
Pages : 8
Date : 2011-07-25
This document describes how IMAP CONVERT can be used within Sieve to
transform messages before final delivery.
A URL for this Internet-Draft is:
http://www.ietf.org/internet-drafts/draft-ietf-sieve-convert-02.txt
I gave this a quick first review. I have the following questions:
1) Do we need a mechanism to discover which conversions are available
(analogous to the IMAP CONVERSIONS command and the
AVAILABLECONVERSIONS data item)?
We can have a new test or magic variables (in a special namespace) for that.
Do you really think that the extra complexity worth it? Maybe you have a
use case in mind.
2) Error handling: what happens when the convert command is issued
with invalid parameters, e.g. unknown conversion, unknown/invalid
parameter etc. ? And what happens when a conversion just fails
temporarily or due to invalid input data from the message?
I think this should be a runtime error.
3) The interaction with subsequent disposition actions and other
convert actions is described explicitly. But what happens with
subsequent tests that evaluate the message body (e.g. header :mime) ?
E.g. for the editheader extension, subsequent header tests all see the
changes made. If tests following the convert command should see the
changes as well, applying the instructed conversions at the end of
Script execution (as suggested at the end of Section 2) will be
difficult.
Hmm. I was mostly thinking about leaf body parts conversions, but other
conversions should be possible.
I agree with applying changed "on the spot" (just like editheader).
_______________________________________________
sieve mailing list
sieve(_at_)ietf(_dot_)org
https://www.ietf.org/mailman/listinfo/sieve