[Top] [All Lists]

Re: [sieve] I-D Action: draft-ietf-sieve-convert-02.txt

2011-08-04 10:09:28
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:
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