This report is correct that the "110" count in the second example (the
first example cited in the report) is incorrect, and should be 99.
The report's second point is a tricky one. I think the report is also
correct about that, and here's why:
Section 1.1 says this:
All interactions transmitted by client and server are in the form of
lines, that is, strings that end with a CRLF. The protocol receiver
of a ManageSieve client or server is either reading a line or reading
a sequence of octets with a known count followed by a line.
That means that each line of the example implicitly includes "CRLF" at
the end of it, and the count includes the final one of those. And
then there has to be another CRLF to end the command itself. In the
first example, that's represented by the empty line, "C:". The second
and third examples lack that.
I intend to resolve this errata report as "Verified". Does anyone on
this list think otherwise?
Barry
On Mon, Nov 30, 2020 at 4:05 AM RFC Errata System
<rfc-editor(_at_)rfc-editor(_dot_)org> wrote:
The following errata report has been submitted for RFC5804,
"A Protocol for Remotely Managing Sieve Scripts".
--------------------------------------
You may review the report below and at:
https://www.rfc-editor.org/errata/eid6345
--------------------------------------
Type: Editorial
Reported by: Kaspar Etter <me(_at_)kasparetter(_dot_)com>
Section: 2.6
Original Text
-------------
Examples:
[…]
C: Putscript "mysievescript" {110+}
C: require ["fileinto"];
C:
C: if envelope :contains "to" "tmartin+sent" {
C: fileinto "INBOX.sent";
C: }
S: OK
C: Putscript "myforwards" {190+}
C: redirect "111(_at_)example(_dot_)net";
C:
C: if size :under 10k {
C: redirect "mobile(_at_)cell(_dot_)example(_dot_)com";
C: }
C:
C: if envelope :contains "to" "tmartin+lists" {
C: redirect "lists(_at_)groups(_dot_)example(_dot_)com";
C: }
S: OK (WARNINGS) "line 8: server redirect action
limit is 2, this redirect might be ignored"
Corrected Text
--------------
Examples:
[…]
C: Putscript "mysievescript" {99+}
C: require ["fileinto"];
C:
C: if envelope :contains "to" "tmartin+sent" {
C: fileinto "INBOX.sent";
C: }
C:
S: OK
C: Putscript "myforwards" {190+}
C: redirect "111(_at_)example(_dot_)net";
C:
C: if size :under 10k {
C: redirect "mobile(_at_)cell(_dot_)example(_dot_)com";
C: }
C:
C: if envelope :contains "to" "tmartin+lists" {
C: redirect "lists(_at_)groups(_dot_)example(_dot_)com";
C: }
C:
S: OK (WARNINGS) "line 8: server redirect action
limit is 2, this redirect might be ignored"
Notes
-----
The octet count of the second example is wrong. Additionally, both the second
and the third example should have an empty client line after the code like
the first example. Otherwise, the octet count of the last example is also
wrong.
Instructions:
-------------
This erratum is currently posted as "Reported". If necessary, please
use "Reply All" to discuss whether it should be verified or
rejected. When a decision is reached, the verifying party
can log in to change the status and edit the report, if necessary.
--------------------------------------
RFC5804 (draft-ietf-sieve-managesieve-09)
--------------------------------------
Title : A Protocol for Remotely Managing Sieve Scripts
Publication Date : July 2010
Author(s) : A. Melnikov, Ed., T. Martin
Category : PROPOSED STANDARD
Source : Sieve Mail Filtering Language
Area : Applications
Stream : IETF
Verifying Party : IESG
_______________________________________________
sieve mailing list
sieve(_at_)ietf(_dot_)org
https://www.ietf.org/mailman/listinfo/sieve