Postgrey Module can't handle comma-separated postfix main.cf

Submitted by KarlMoik on Mon, 03/21/2011 - 02:53

Hey guys,

small but annoying bug: When I have the following line in my postfix main.cf:

smtpd_recipient_restrictions = reject_unauth_pipelining, reject_non_fqdn_recipient, reject_unknown_recipient_domain, check_policy_service inet:127.0.0.1:10031, permit_sasl_authenticated, reject_unauth_destination, check_policy_service inet:127.0.0.1:12525, permit_mynetworks, check_policy_service unix:/var/spool/postfix/postgrey/socket, permit

the Virtualmin -> EMail Messages -> EMail Greylisting claims that greylisting is disabld. After clicking the enable button it changes the line to:

smtpd_recipient_restrictions = reject_unauth_pipelining, reject_non_fqdn_recipient, reject_unknown_recipient_domain, check_policy_service inet:127.0.0.1:10031, permit_sasl_authenticated, reject_unauth_destination, check_policy_service inet:127.0.0.1:12525, permit_mynetworks, check_policy_service unix:/var/spool/postfix/postgrey/socket permit

which is basically the same but w/o the last comma (before the last "permit". Not serious but I prefere the commas with such long rules ;)

Thank you and regards

Status: 
Closed (fixed)

Comments

Submitted by JamieCameron on Mon, 03/21/2011 - 12:56

Thanks for pointing this out - I will fix that bug in 3.85 Virtualmin release.

Submitted by Issues on Mon, 04/04/2011 - 14:20

Automatically closed -- issue fixed for 2 weeks with no activity.