1 (edited by Jeronymus 2014-05-06 22:39:09)

Topic: Per User/Per Domain Greylisting (opt-in/opt-out) not working

======== Required information ====
- iRedMail version: 0.8.6
- Store mail accounts in which backend (LDAP/MySQL/PGSQL): LDAP
- Linux/BSD distribution name and version: Debian Wheezy (7.5)
- Related log if you're reporting an issue:
====

If I try to change the enabled service Greylisting in the Advanced Tab for a specific user or even for an entire domain, the checkbox keeps the previous state after saving the changes. (ie if the checkbox was checked and I uncheck it, after saving it is checked again)
Also if I look in database postfixpolicyd in table policy no rows are added or removed.

This issue is reproducible in the LDAP backend demo version (https://demo.iredmail.org/iredadmin-pro-ldap/). 
Also tried in MYSQL backend demo and this works as expected.

2

Re: Per User/Per Domain Greylisting (opt-in/opt-out) not working

I will try to reproduce this issue and come back to you later. Thanks for your feedback.

3

Re: Per User/Per Domain Greylisting (opt-in/opt-out) not working

Bug confirmed. It's caused by incorrect function name. You can fix it with below steps:

1: Open file libs/policyd/greylist.py, find below line:

def setGreylistingStatus(self, account, enabled=True):

2: Change name "setGreylistingStatus" to "set_greylisting_status" like below:

def set_greylisting_status(self, account, enabled=True):

3: Save your change and restart Apache service.

That's it.

Thanks again for your feedback. This fix will be available in next release of iRedAdmin-Pro.

4

Re: Per User/Per Domain Greylisting (opt-in/opt-out) not working

Thanks for your patch, works like a charm!