Hans Sandsdalen wrote:
I added these lines

   if ( $transaction->{_relaying} || exists($ENV{RELAYCLIENT}) )

This code will not work for several reasons:

1) There is no '$transaction->{_relaying}' attribute (don't ever look inside the object outside of that package);

2) The relay_client() method has been part of the $connection object since 0.29 was released.

Since you didn't say what version of Qpsmtpd you are running, I don't know what exactly to suggest. You should upgrade to 0.29 at least (if not the repository trunk), where the correct line would be

        if ( $connection->relay_client() || exists($ENV{RELAYCLIENT}) )

although I still prefer the relay_ok plugin I suggested, rather than patching all other connect plugins to respect the relaying status (I'll test and check this in today if I have the chance).

HTH

John

Reply via email to