Hi Roundcube devs,

We have been using Roundcube since many years and migrating from the unmaintained SieveRules plugin by JohnDoh to the official ManageSieve plugin. There are a few things that we want to improve, and we would like to know if it is okay to send these as patches so they can be included in a later official version of the Managesieve plugin.

1. Import of disabled rules generated by the SieveRules plugin

ManageSieve uses the trick "if false ..." to disable sieve rules, but SieveRules stores disabled rules as a PHP serialized form of it's internal representation in a sieve comment. Now we have a working version for this that is able to import conditions and vacation (autoreply) rules

2. Add original subject to vacation replies

SieveRules can add the original subject to the automatically sent reply, by using a trick of creating a ${subject} sieve variable in a separate rule that is hidden in SieveRules, but ManageSieve imports is an unnamed rule. The rule looks like this:

set "subject" ""; if header :matches "subject" "*" {     set "subject" "${1}"; }

There is also a feature request for this: https://github.com/roundcube/roundcubemail/issues/5258

Our plan is to support this extra rule and also hide it from the UI and add a checkbox to Subject in the vacation part that would append ${subject}. The appended ${subject} would be also hidden from the UI only the checkbox would be used to indicate if the original subject is added to the reply or not. The extra rule would be inserted only if at least one vacation rule wants to append the original subject.

We would also add extra configuration options to the plugin, so both features could be disabled by default, especially for the second one, as many considers this as a source of spamming (however big email providers, like GMail appends the old subject when sending an autoreply)

So, I'm waiting for your comments, suggestions on the features or if the are applicable for inclusion in the official version

Thanks in advance, Daniel