[Tickets #14397] Ingo Vacation Setting Produces invalid .procmailrc

[noreply at bugs.horde.org]

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

Ticket URL: https://bugs.horde.org/ticket/14397
------------------------------------------------------------------------------
  Ticket             | 14397
  Created By         | canderson at foxtwo.net
  Summary            | Ingo Vacation Setting Produces invalid .procmailrc
  Queue              | Ingo
  Version            | 3.2.10
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


canderson at foxtwo.net (2016-06-12 19:49) wrote:

When enabling a vacation message with Ingo set up to use procmail, the  
resulting .procmailrc has an error that causes messages to bounce with  
the message:

procmail: Extraneous deliver-head flag ignored

Looking at the generated .procmailrc I see:

...
      :0 h
      SUBJECT=| formail -xSubject:

      :0 Whc: ${VACATION_DIR:-.}/vacation.lock
      * ? test $DATE -gt $START && test $END -gt $DATE
      {
        :0 Wh
...

According to the procmail instructions, an "h" is invalid when the  
action is to open braces {}. I made a n edit to  
/ingo/lib/Script/Procmail/Recipe.php to remove the "h" flag:

185c185
<                     '    :0 Wc: ${VACATION_DIR:-.}/vacation.lock';
---
>                     '    :0 Whc: ${VACATION_DIR:-.}/vacation.lock';

It now works as expected. I haven't tested all combinations of  
settings, so I am unsure if that will break something else, but it  
does produce a valid file.