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

Review of draft-ietf-sieve-refuse-reject-02

2006-08-01 16:39:07

(1) The document should make clear in the Abstract that it is updating the existing "Reject" action.

(2) I suggest adding "The message is rejected after end of data" to the end of the Abstract.

(3) Typo in Abstract and also Section 1: "updates definition" should be "updates the definition".

(4) Suggested change in Section 1: change "sometimes preferable" to "generally preferable".

(5) Typos (grammatical errors) in Section 3.1:

Old:
   How message is refused depends
   on capabilities of mail component (MUA, MDA or MTA) executing the
   Sieve script. The Sieve interpreter must do one of the following
   actions, as detailed by the following priority table (items listed
   earlier take precedence). Note that if action can not be taken or
   fails, the interpreter should try the next item in the list:

New:
   How a message is refused depends
   on the capabilities of the mail component (MUA, MDA or MTA) executing the
   Sieve script. The Sieve interpreter must do one of the following
   actions, as detailed by the following priority table (items listed
   earlier take precedence). Note that if an action can not be taken or
   fails, the interpreter should try the next item in the list:

(6) What is the rational for item 1 on the action list in Section 3.1 (as opposed to rejecting even if the MAIL FROM is null)?

(7) Typo in Section 3.1.1: "at SMTP/LMTP level" should be "at the SMTP/LMTP level".

(8) My suggestion for the open issue in 3.1.1 would be to require ASCII, since it is the most interoperable. A future extension can permit UTF-8 after wider deployment of EAI extensions. That means Sieve should enforce ASCII in the use of 'Reject'. Replacing UTF-8 with "?" is a kludge that is likely to confuse users and lead to undesirable results. Making it a syntax error seems the way to go, as users will be immediately notified, and clients that generate Sieve scripts can guide users.

(9) Typo in Section 3.1.1: "delay immediate sending" should be "delay immediately sending".

(10) Typo in Section 3.1.3: "When Sieve engine is running inside MUA" should be "When the Sieve engine is running inside the MUA".

(11) Missing blank line in Section 3.1.3, before sentence starting "MTAs and MDAs SHOULD NOT".

(12) Typo in Section 3.1.3: "reject at protocol level" should be "reject at the protocol level".

(13) In Section 3.2, should there be any discussion of priority, for example, if both vacation and reject are attempted, do the reject and not the vacation?





--
Randall Gellens
Opinions are personal;    facts are suspect;    I speak for myself only
-------------- Randomly-selected tag: ---------------
Premature optimization is the root of all evil
                             --C. A. R. Hoare