Missing information in 40-policy_banks

Bug #1963796 reported by damluk
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
amavisd-new (Ubuntu)
Triaged
Medium
Unassigned

Bug Description

I have been looking at the difference between the Debian and Ubuntu source packages and noticed 40-policy_banks.

Said config file does not do anything on its own. The included @author_to_policy_bank_maps assign policybanks which do not exist, and there is no Ubuntu shipped documentation other than the comment in the header.

I assume that the original Ubuntu package maintainer did not see the functional dependency to the policy_bank definitions in the example of the upstream amavisd-new-2.6.0 release notes.

I think there should be an additional explanation that these policybanks have to be created as well.

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks for taking the time to file this bug and try to make Ubuntu better.

After taking a look at the mentioned file (debian/etc/conf.d/20-policy_banks in the source package), it seems that you are right, this config has no effect since there is no definition of `$policy_bank{'WHITELIST'}`. In order to have something that would minimally work (not sure if this is the original expected behavior) we would need to add the following:

  $policy_bank{'WHITELIST'} = {
    bypass_spam_checks_maps => [1],
    spam_lovers_maps => [1],
  };

This is the snippet provided by upstream in the release notes. I am not a amavisd-new expert but I agree with your assessment.

I am marking this bug as Triaged, setting the importance to Medium and subscribing ubuntu-server.

Changed in amavisd-new (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.