Milter-greylist

Hi, postfix sets the hostname of clients without a reverse dns mapping to `unknown -- maybe milter-greylist should do the same: postfix/smtpd[15288]: connect

Petar Bogdanovic <petar@...> wrote:

> Patch is attached. Once applied, clients without a reverse dns mapping
> will trigger the SpamAssassin RDNS_NONE test.

Sendmail logs are a bit different, and are inconsistent. Anyone has an
opinion whether this should be ifdef POSTFIX, or enabled for both
Postfix and Sendmail?
-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@...

On Tue, Dec 30, 2008 at 06:38:33AM +0100, manu@... wrote:
> Petar Bogdanovic <petar@...> wrote:
> 
> > Patch is attached. Once applied, clients without a reverse dns mapping
> > will trigger the SpamAssassin RDNS_NONE test.
> 
> Sendmail logs are a bit different, and are inconsistent. Anyone has an
> opinion whether this should be ifdef POSTFIX, or enabled for both
> Postfix and Sendmail?

It depends on what Sendmail passes to mlfi_connect(ctx, hostname, addr)
as hostname. If it's the same like Postfix -- [some.ip.addr.ess] -- then
enabling it for both MTAs would make no difference IMHO.

I don't have Sendmail so I can't check this, but basically one just has
to dig through the logs and find a Sendmail session with a no-rdns
client. The interesting part are the milter-greylist lines then:

   milter-greylist: (unknown id): addr some.host.name[some.ip.addr.ess] (...)

Petar Bogdanovic <petar@...> wrote:

> postfix sets the hostname of clients without a reverse dns mapping to
> `unknown' -- maybe milter-greylist should do the same:

Some people do check for IP without reverse DNS using a regex right now.
As far as I understand, your change will break their config.

I can include it, but only if it is activated by a config-file option.

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@...