[exim-dev] [Bug 1057] Patch: Allow multiple TCP clamd scanni…

Top Page
Delete this message
Reply to this message
Author: Todd Lyons
Date:  
To: exim-dev
Old-Topics: [exim-dev] [Bug 1057] New: Patch: Allow multiple TCP clamd scanning servers
Subject: [exim-dev] [Bug 1057] Patch: Allow multiple TCP clamd scanning servers
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=1057




--- Comment #15 from Todd Lyons <tlyons@???> 2012-11-10 23:25:27 ---
(In reply to comment #14)
> av_scanner = clamd:192.0.2.3 1234:local
>
> The third one could be regarded as an option for clamd or as a modifier for the
> specific address (there being only one). How does the patched code play into
> this? Will a "local" affect one all addresses, or one (which)? Is that what
> is wanted?


I recall noticing that 'local' was moved and it still does look for it and
accept it.

> It's slightly unfortunate that the proposed syntax looks like a list, but is
> actually a command plus a list - and using the same separator for the list as
> between the command and the list. Is there any scope for a more clear
> separation? What will happen if additional non-address options are needed in
> future?


I think that a comma should be used to pass those additional options instead of
a space. The semi-colon could be too easily confused with the colon when
looking at it.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email