From: Ralf Hildebrandt on
* Noel Jones <njones(a)megan.vbhcs.org>:

> I believe Ralf's request is about the smtp rejection message sent to
> the remote client, not about postfix logging.

Yes.

> ie. the current reject response in postscreen.c around line 920 or
> so looks something like:
>
> if (dnsbl_action == PS_ACT_DROP) {
> smtp_reply(vstream_fileno(state->smtp_client_stream),
> state->smtp_client_addr, state->smtp_client_port,
> "521 5.7.1 Blocked by DNSBL\r\n");
> state->flags |= PS_FLAG_NOFORWARD;
> }
>
> Often complaints are reported by a remote customer forwarding the
> reject message by an alternate channel.

Exactly.

> It would be easier to track down customer complaints if the reject
> message contained "521 5.7.1 Client 192.0.2.1 Blocked by DNSBL"

That's exactly the form I'd like to see. It doesn't even have to
mention the DNSBL used. Just the IP!

--
Ralf Hildebrandt
Geschäftsbereich IT | Abteilung Netzwerk
Charité - Universitätsmedizin Berlin
Campus Benjamin Franklin
Hindenburgdamm 30 | D-12203 Berlin
Tel. +49 30 450 570 155 | Fax: +49 30 450 570 962
ralf.hildebrandt(a)charite.de | http://www.charite.de


From: Noel Jones on
On 6/16/2010 8:00 PM, Wietse Venema wrote:

[discussion about changing SMTP reject message]

>
> It will involve more change, as smtp_reply() currently does not
> take a format string. All smtp_reply() calls would need to be
> replaced by smtp_reply(handle, addr, port, "%s", text). The
> alternative, a second smtp_reply() function that does take a format
> string, would be asking for format string errors.
>
> Wietse

Thanks.

Looking forward to the enhanced logging.


-- Noel Jones