[Tickets #8105] Re: Driver Timsieve saves invalid sieve file without displaying error message

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8105
------------------------------------------------------------------------------
  Ticket             | 8105
  Updated By         | matthias.andree at uni-paderborn.de
  Summary            | Driver Timsieve saves invalid sieve file without
                     | displaying error message
  Queue              | Ingo
  Version            | 1.2.1
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


matthias.andree at uni-paderborn.de (2009-03-20 11:42) wrote:

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.