Christian Roessner:
> Hi,
> 
> > This is uploaded as postfix-2.8-20101230-nonprod. The code has had
> > limited testing, so keep an eye on things if you intend to expose
> > it to the network.
> 
> Minor questions:
> 
> postfix/tlsproxy[30864]: CONNECT [2a01:4f8:120:31e2::165]51824

Right. This is the result from a late change while cleaning up
the internal protocols.

> It is just, because I saw it: Missing ":" between address and
> port. And by the way: For postscreen and dnsblog and ... are you
> planning on making these modules behave like $smtpd_client_port_logging
> (default: no) ?

This is not an option. postscreen(8) and tlsproxy(8) programs MUST
log the client port number, otherwise their logging becomes
incomprehensible. With smtpd(8), you can still distinguish between
different sessions from the same client by the smtpd(8) process ID.

> And one minor thing. When rebuilding Ubuntu packages for the
> 20101230-nonprod, I reviewed master.cf for the tlsproxy line. I
> added it with a comment sign, read from the POSTSCREEN_README. The
> current master.cf is missing it.

The POSTFIX post-install procedure adds the missing line. If UBUNTU
does not use the POSTFIX post-install procedure, then they deserve
the pain of having to duplicate its functionality. This includes
duplicating the code that adds lines that weren't present in
previously installed configuration files, and duplicating the code
that installs and sets permissions on files (e.g. executables or
manpages) that were not present in previous Postfix versions.

        Wietse

> So far, hope you don't mind my little comments :-)
> 
> I wish you all a good change from old->new year.
> 
> Christian
> 
> ---
> Roessner-Network-Solutions
> Bachelor of Science Informatik
> Nahrungsberg 81, 35390 Gie?en
> F: +49 641 5879091, M: +49 176 93118939
> USt-IdNr.: DE225643613
> http://www.roessner-network-solutions.com
> 
> 
> 

Reply via email to