[JSIEVE-75] incorrect parsing of script containing multiple multi-line forms

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 0.4
• Fix Version/s: 0.5
• Component/s: JSieve (Main)
• Assignee: Stefano Bagnara
• Reporter: Vishal Mahajan
• Created: 15/Mar/11 13:06
• Updated: 31/Mar/11 12:37
• Resolved: 31/Mar/11 10:57

Description

I am using jSieve 0.4. I have the following sieve script containing two multi-line forms:

---

require ["fileinto", "reject", "tag", "flag"];

1. filter1
   if anyof (header :contains "subject" "abc") {
   reply text:
   reply1
   .
   ;
   reply text:
   reply2
   .
   ;
   }

   ---

The issue is that the jSieve parser sees only one multi-line form in the above script, with form data being the text between the first "text:" and last "CRLF.CRLF", i.e.

---

reply1
.
;
reply text:
reply2

---

Is there an issue in the sieve.jjt grammar in dealing with multiple multi-line forms?

One of the odd things I observed in sieve.jjt is that NEWLINE is considered as "\n"|"\r"|"\r\n", whereas RFC 3028 (section 2.4.2) requires that a multi-line form:

starts with the keyword "text:",
followed by a CRLF, and ends with the sequence of a CRLF, a single
period, and another CRLF.
which implies that standalone "\n" and "\r" chars should not be considered in determining start and end of multi-line form data.