voxpupuli / puppet-postfix

Puppet postfix module
Apache License 2.0
70 stars 174 forks source link

main.cnf.erb always sets postgrey #263

Closed jamesj2 closed 3 years ago

jamesj2 commented 4 years ago

The following conditional block in main.cnf.erb always defaults to check_policy_service unix:postgrey/socket, even when the postgrey option is set to false.

<% if !@smtpd_recipient_restrictions.empty? or @postgrey -%>
smtpd_recipient_restrictions =
<% @smtpd_recipient_restrictions.each do |line| -%>
  <%= line %>,
<% end -%>
<% if @postgrey_policy_service -%>
  check_policy_service <%= @postgrey_policy_service %>,
<% else -%>
  check_policy_service unix:postgrey/socket,
<% end -%>

End result:

smtpd_recipient_restrictions =
  permit_mynetworks,
  reject_unauth_destination,
  check_policy_service unix:postgrey/socket,
stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.