Skip to content

Implement basic sieve support using Dovecot.#166

Merged
tomav merged 3 commits intodocker-mailserver:v2from
stonemaster:basic-sieve-support
Apr 28, 2016
Merged

Implement basic sieve support using Dovecot.#166
tomav merged 3 commits intodocker-mailserver:v2from
stonemaster:basic-sieve-support

Conversation

@stonemaster
Copy link
Copy Markdown
Contributor

The dovecot-sieve plugin is installed and configured to apply sieve
as soon as a .dovecot.sieve file is encountered in the virtual user's
home directory (that is /var/mail/${domain}/${username}/.dovecot.sieve).

Transport has been changed in the postfix configuration to use
Dovecot LDA (see http://wiki.dovecot.org/LDA/Postfix) to actually
enable sieve filtering.

Tests have been added.

Addresses #87.

The dovecot-sieve plugin is installed and configured to apply sieve
as soon as a .dovecot.sieve file is encountered in the virtual user's
home directory (that is /var/mail/${domain}/${username}/.dovecot.sieve).

Transport has been changed in the postfix configuration to use
Dovecot LDA (see http://wiki.dovecot.org/LDA/Postfix) to actually
enable sieve filtering.

Tests have been added.
@stonemaster
Copy link
Copy Markdown
Contributor Author

I didn't add documentation specifically. I thought about adding a new entry to the FAQ on how to get started with sieve.

Comment thread test/tests.bats Outdated
# sieve
#

@test "checking sieve: user1 should have received 1 in folder INBOX.spam" {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should have received 1 email

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Apr 28, 2016

Thank you @stonemaster, I just let you fix the missing word in the test.
I'll merge as soon it's fixed.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Apr 28, 2016

Regarding the doc, yes a page Configure Sieve filters in the Wiki will be fine with an example and a link to documentation.

@00angus
Copy link
Copy Markdown
Contributor

00angus commented Apr 28, 2016

cool. It could be useful to be able to provide user-prepared sieve files to be copied in place during setup.
sort of: [email protected] if found during startup will be moved into corresponding home into .dovecot.sieve.
What do you think about it? Other ideas?
BTW I was working on the same thing but using lmtp instead of lda... performance wise do you see differences?

@stonemaster
Copy link
Copy Markdown
Contributor Author

@00angus that's a nice idea. I would suggest have a differen pull request for that feature though.
Regarding LMTP: I have no idea whether this actually makes a difference. LDA is the older (maybe proven?) method. Have you seen any performance tests regarding this?

@stonemaster
Copy link
Copy Markdown
Contributor Author

@tomav Added a new Wiki page and updated README with a link to it btw.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Apr 28, 2016

Regarding user specific config, let's open another issue / enhancement.

@tomav tomav merged commit 570bf85 into docker-mailserver:v2 Apr 28, 2016
@00angus
Copy link
Copy Markdown
Contributor

00angus commented Apr 28, 2016

LMTP: needed in case smtp wants to deliver to a different host/container, better descriptive information in case of failures than LDA, LMTP may add additional headers I think ...
Performance wise, on high email load I think LMTP should perform better as it's a running process sitting in the background waiting for connections: on the contrary LDA will need to spawn a delivery process for each new message.
Anyway both solutions for me work :)

If performance becomes a problem for someone please let us know ;)

RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
…-support

Implement basic sieve support using Dovecot.
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

Successfully merging this pull request may close these issues.

3 participants