Skip to main content

Last Call Review of draft-ietf-extra-sieve-special-use-04
review-ietf-extra-sieve-special-use-04-secdir-lc-farrell-2018-12-17-00

Request Review of draft-ietf-extra-sieve-special-use
Requested revision No specific revision (document currently at 05)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2018-12-20
Requested 2018-12-06
Authors Stephan Bosch
Draft last updated 2018-12-17
Completed reviews Genart Last Call review of -04 by Meral Shirazipour (diff)
Secdir Last Call review of -04 by Stephen Farrell (diff)
Opsdir Last Call review of -04 by Shwetha Bhandari (diff)
Assignment Reviewer Stephen Farrell
State Completed
Review review-ietf-extra-sieve-special-use-04-secdir-lc-farrell-2018-12-17
Reviewed revision 04 (document currently at 05)
Result Ready
Completed 2018-12-17
review-ietf-extra-sieve-special-use-04-secdir-lc-farrell-2018-12-17-00
As stated in the draft I see no new security issues here, given that SIEVE
already includes the ability to deliver messages and create mailboxes.

A non-security comment, that you should feel free to ignore: as a mail user it
is irritating to have multiple folders (that may or may  not have a 
"special-use" attribute associated with 'em) created by multiple MUAs and into
which messages can be deposited in a manner that's unpredictable to me as a
user . I understand that this spec is already trying to help with that, by
reducing the likelihood that such redundant folders get created. But I already
have a bunch of those folders, so while this is likely the wrong place for such
text, I'd have been happier if the guidance for what to do when there's >1
matching special-use mailbox had been more deterministic, even when the
SIEVE-supplied default one doesn't exist. For example, if you'd been able to
say to sort the matching folders in some way and then pick the first that'd be
better I think, but maybe there are reasons why you can't do that in SIEVE.
(I'd be even happier if someone was working on specs/tooling to help merge all
those folders somehow, but that's definitely out of scope here.)