Re: [sieve] [Editorial Errata Reported] RFC5804 (6345)

[Barry Leiba <[e-mail]>]

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
<[e-mail]> 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 <[e-mail]>
>
> 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 "[e-mail]";
>        C:
>        C: if size :under 10k {
>        C:     redirect "[e-mail]";
>        C: }
>        C:
>        C: if envelope :contains "to" "tmartin+lists" {
>        C:     redirect "[e-mail]";
>        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 "[e-mail]";
>        C:
>        C: if size :under 10k {
>        C:     redirect "[e-mail]";
>        C: }
>        C:
>        C: if envelope :contains "to" "tmartin+lists" {
>        C:     redirect "[e-mail]";
>        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