Re: [exim-dev] tls_advertise_hosts issues too many warnings …

Top Page
Delete this message
Reply to this message
Author: Jeremy Harris
Date:  
To: exim-dev
Subject: Re: [exim-dev] tls_advertise_hosts issues too many warnings (!?)
On 18/11/15 06:23, Heiko Schlittermann wrote:
> Jeremy Harris <jgh@???> (Di 17 Nov 2015 15:02:25 CET):
>> On 17/11/15 13:02, Heiko Schlittermann wrote:
>>>     exim -C /dev/null -bP primary_hostname
>>>     exim -C <(echo primary_hostname=foo) -bP primary_hostname

>>>
>>> With the default being tls_advertise_hosts = *, this above line issue
>>> all that ugly warnings about missing tls_* support files.
>>
>> How about suppressing the tls warnings when -bP is used?
>
> Ok, I've implemented this, using the existing 'checking' variable in exim.c.
> Now I set this in -bP and -brw too.


not -brt ?

and...
>
> But, ~ exim.c:3490
>
>     if …
>      (
>      list_options &&
>      (checking || smtp_input || extract_recipients ||
>        filter_test != FTEST_NONE || bi_option)
>      ) ||
>      … then issue "incompatible options"

>
> Are there security reasons, or may I change this to
>
>     if …
>      (
>      list_options &&
> -    (checking || smtp_input || extract_recipients ||
> +    (smtp_input || extract_recipients ||
>        filter_test != FTEST_NONE || bi_option)
>      ) ||
>      … then issue "incompatible options"


no, that would not be good; the combination "-be -bP" would
then be allowed, and it makes no sense.

I think you need (list_options || list_config) instead there,
and don't set "checking" for -bP.

--
Cheers,
Jeremy