search

prehor / amavisd-milter

Milter interface for the amavis spam filter engine.
BSD 3-Clause "New" or "Revised" License
8 stars 4 forks source link

Refactor client_name handling in mlfi_connect #4

Closed prehor closed 5 years ago

prehor commented 5 years ago

Spamassassin's RDNS_NONE check requires in the synthesized Received: header Forward-confirmed reverse DNS (FCrDNS) client name or "unknown" if the FCrDNS is not available.

Sendmail sends the reverse DNS or IP address in square brackets both in the client_host argument and the {client_name} macro. If the client name is FCrDNS, macro {client_reverse} is set to OK, otherwise to FORGED, TEMP or FAIL.

Postfix sends a FCrDNS or "unknown" using the {client_name} macro.

Related-to: #1, #3

Signed-off-by: Petr Řehoř rx@rx.cz

prehor commented 5 years ago

@Bluewind, thanks for your hint about git rebase. Now this PR looks better.

Could you please try that it works correctly with the postfix? I can only test it with a sendmail.

Bluewind commented 5 years ago

It appears to work correctly for me with postfix. Thanks!