Author: Jasen Betts Date: To: exim-dev Subject: Re: [exim-dev] making transport_filter conditional
On 2013-10-07, Jeremy Harris <jgh@???> wrote: > On 05/10/13 21:12, Jasen Betts wrote:
>> currently (unless it's been changed since 4.80)
>> there's no way to use expansion to disable the transport_filter
>> transport option.
>
> transport_filter is an expanded string. You should be able
> to have it expand to an empty string when you don't
> want a filter.
The problem is the code doesn't check for an empty string
I changed this:
if (transport_filter_argv == NULL)
to this:
if (transport_filter_argv == NULL || ! *transport_filter_argv || ! **transport_filter_argv )
to get that effect.
However it might make more sense to move the test for an empty string to the place
where transport_filter is being expanded and transport_filter_argv is being set,
as then it could avoid expanding the rest of the values.
