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

delay_deny: add included mode #1564

Merged
merged 1 commit into from Aug 17, 2016

Conversation

Projects
None yet
2 participants
@gramakri
Collaborator

gramakri commented Aug 12, 2016

Currently, this plugin delay denies everything unless listed in the
config file. This can be a little risky when experimenting this plugin
when a lot of other plugins are already in use.

In such cases, it's easier to use an included mode (a whitelist mode)
where one can add plugins to bypass one by one.

delay_deny: add included mode
Currently, this plugin delay denies everything unless listed in the
config file. This can be a little risky when experimenting this plugin
when a lot of other plugins are already in use.

In such cases, it's easier to use an included mode (a whitelist mode)
where one can add plugins to bypass one by one.
@gramakri

This comment has been minimized.

Show comment
Hide comment
@gramakri

gramakri Aug 15, 2016

Collaborator

Made this patch when investigating #1558 based on @smfreegard 's suggestion.

@smfreegard @msimerson @baudehlo comments?

Collaborator

gramakri commented Aug 15, 2016

Made this patch when investigating #1558 based on @smfreegard 's suggestion.

@smfreegard @msimerson @baudehlo comments?

@smfreegard smfreegard merged commit 57d3fb4 into haraka:master Aug 17, 2016

2 checks passed

codecov/project 36.12% (+0.00%) compared to cdad67d
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment