Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incomplete fix for bug #145 #172

Open
JKingweb opened this issue Jan 17, 2017 · 2 comments
Open

Incomplete fix for bug #145 #172

JKingweb opened this issue Jan 17, 2017 · 2 comments

Comments

@JKingweb
Copy link

JKingweb commented Jan 17, 2017

Version 1.9.2 is supposed to have allowed empty lists per #145, but this does not seem to have been universally applied. The whitelist under the greylisting section when using rmilter 1.10.0 does not accept such an empty list, and I get a parse error:

bind_socket = inet:2112@localhost;

spamd {
	whitelist = ;
};

greylisting {
	whitelist = ;
};

With the following error in syslog:

rmilter[23449]: config file </etc/rmilter.conf.d/rmilter.conf> parse error! line: 8, text: ;, reason: syntax error

@vstakhov
Copy link
Owner

Don't use greylisting in Rmilter.

@JKingweb
Copy link
Author

I wasn't planning on it. I discovered this because, as a first-time user of rspamd, I wanted to have empty whitelists for everything in rmilter.conf.common during rudimentary testing (as my test messages were getting whitelisted and rspamd was never running), and I happened to put the greylisting section first, initially. Thus I spent time trying to figure out what was wrong, why the documentation led me astray, whether this bug had been reported, whether it was a regression... It would have saved me a lot of trouble had the allowed syntax simply been consistent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants