vacation problem with SRS

[Stephan Bosch]

Op 7/20/2017 om 8:24 PM schreef Kacper Guzik:
> I have this version with FIXME
>
>
>         /* FIXME: If From header of message has same address, we should use
> that
>
>          * instead to properly include the phrase part.
>
>          */
>
>         rfc2822_header_printf(msg, "To", "<%s>", reply_to);
>
>
>
> This should be work ok? Or You must change something?
>
> 2017-07-20 15:51 GMT+02:00 Stephan Bosch <stephan at rename-it.nl>:
>
>>
>> Op 20-7-2017 om 10:37 schreef Kacper Guzik:
>>
>> Hi i have similiar problem like here:
>>> http://www.iredmail.org/forum/topic11833-iredmail-support-vo
>>> cation-respone-unknown-user.html
>>>
>>>
>>> email send :
>>> from : web123 at gmail.com
>>> to : jacka at mail.com
>>>
>>> vocation sent back
>>> from : jacka at mail.com
>>> to : srs0=hmc8=v7=gmail.com=web123 at mail.com
>>>
>>>
>>>
>>> postsrsd changing return-path from web123 at gmail.com to srs0=hmc8=v7=
>>> gmail.com=web123 at mail.com
>>>
>>> this is no problem for me but  sieve:
>>>
>>> Vacation's messages are always addressed to the Return-Path address
>>>
>>> it is possible some how change this ugly to header to normal?
>>>
>>> I can't find anything on dovecot mailing lists
>>>
>> Hmm,
>>
>> I think this relates to this FIXME:
>>
>> https://github.com/dovecot/pigeonhole/blob/master/src/lib-
>> sieve/plugins/vacation/cmd-vacation.c#L951
>>
>> Looks like I've been just lazy. Should be relatively easy to fix.

I addressed the FIXME earlier already and I have this pending now:

https://github.com/dovecot/pigeonhole/commit/0e5ec449f701bdc293c5fdc2551d0a7b242703d7

Regards,

Stephan.