Tickets :: Comment on [#8105] sieve driver doesn't check header field validity

* Your Email Address
* Spam protection	Enter the letters below: __.. .. .. ..___. (__ \|\/\|\|__\|\| \| _/ .__)\| \|\| \|\|/\\|./__.
Comment	> To detail on the problem, (a) Ingo generates a rule that Cyrus's > timsieved will reject (it expects header name matches without colon, > as Heiko already stated). (b) The attempt to upload this invalid > Sieve script fails SILENTLY, without error message, while Cyrus > continues with the previous edition of the script. (c) the user > interface in however presents the new filter. > > > > The fix should be two-fold: (a) fix Ingo so it does not create > invalid rules in the first place (stripping the trailing colon should > be in order for user-specified header filters, otherwise it should > complain), (b) - suggested by Heiko above - make sure to display the > error if the Sieve script cannot be uploaded to the server. > >
Attachment
Watch this ticket