Skip to main content

Sieve Email Filtering: Body Extension
RFC 5173

Yes

(Lisa Dusseault)

No Objection

Lars Eggert
(Chris Newman)
(Dan Romascanu)
(David Ward)
(Jon Peterson)
(Magnus Westerlund)
(Mark Townsley)
(Ron Bonica)
(Ross Callon)
(Tim Polk)

Note: This ballot was opened for revision 09 and is now closed.

Lars Eggert
No Objection
Lisa Dusseault Former IESG member
Yes
Yes ()

                            
Chris Newman Former IESG member
No Objection
No Objection ()

                            
Cullen Jennings Former IESG member
No Objection
No Objection (2008-01-10)
I like this one.
Dan Romascanu Former IESG member
No Objection
No Objection ()

                            
David Ward Former IESG member
No Objection
No Objection ()

                            
Jari Arkko Former IESG member
No Objection
No Objection (2008-01-10)
Review by Christian Vogt:

Draft-ietf-sieve-body-07 extends the Sieve email processing language by
a command for searching a text string in the body of an email.

The document is concise and (with few exceptions mentioned below) clear,
and should therefore considered ready for publication once the comments
below have been addressed.


Conceptual:

Section 3 specifies that a search for an empty string should return
"false" for emails that do not have a body, and "true" for emails that
have an empty body.  I believe that such different handling of similar
inputs (no body vs. empty body) may be confusing for users who are not
Sieve experts, and hard to debug for Sieve expert.  Unless the search
result cannot be the same for both cases for substantial reasons, the
search results should be redefined to be the same.  If there is
substantial reasons for the search results to be different, then the
draft should provide rationale for such distinction.


Editorial:

(1)  Section 2, 3rd paragraph:  What is a "capability string"?  If this
is an expression already defined in [SIEVE], then [SIEVE] should be
cited near the occurrence of "capability string".  Otherwise,
"capability string" should be defined in this document.

(2)  Section 4.3, 2nd paragraph:  Unstated what the difference between a
body transform of ':text' and a body transform of ':content "text"' is.
 The difference can be derived by the reader, of course, but the
document is incomplete by leaving it unstated.

(3)  Section 5, 2nd paragraph:  s/wild card/wildcard/ and s/set
match/set-match/
Jon Peterson Former IESG member
No Objection
No Objection ()

                            
Magnus Westerlund Former IESG member
No Objection
No Objection ()

                            
Mark Townsley Former IESG member
No Objection
No Objection ()

                            
Ron Bonica Former IESG member
No Objection
No Objection ()

                            
Ross Callon Former IESG member
No Objection
No Objection ()

                            
Russ Housley Former IESG member
No Objection
No Objection (2008-01-07)
  From Gen-ART Review by Elwyn Davies; however, I think this is even
  more important than indicated by Elwyn.  Elwyn said:

  A pointer is given to the SIEVE base document, but I wonder if this
  document should note that ABNF is used to specify syntax and provide
  the appropriate reference.
Tim Polk Former IESG member
(was No Record, Discuss) No Objection
No Objection ()