Skip to content

Add sieve pipe and filter#574

Merged
tomav merged 7 commits intodocker-mailserver:masterfrom
arneke:add-sieve-pipe-and-filter
May 10, 2017
Merged

Add sieve pipe and filter#574
tomav merged 7 commits intodocker-mailserver:masterfrom
arneke:add-sieve-pipe-and-filter

Conversation

@arneke
Copy link
Copy Markdown
Contributor

@arneke arneke commented Apr 15, 2017

Ref #505 , I have added support for sieve_extprograms as non-global plugin, i.e. the users can add the whitelisted programs to their personal Sieve files.

The whitelisted programs are copied from config/sieve-pipe and config/sieve-filter to /usr/lib/dovecot/sieve-pipe and /usr/lib/dovecot/sieve-filter

I have added target/dovecot/90-sieve.conf, even though it really just changes the following lines:

  • sieve_plugins = sieve_extprograms
  • sieve_extensions = +notify +imapflags +vnd.dovecot.pipe +vnd.dovecot.filter
    and adds
  • sieve_pipe_bin_dir = /usr/lib/dovecot/sieve-pipe
  • sieve_filter_bin_dir = /usr/lib/dovecot/sieve-filter

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Apr 17, 2017

Restarted build.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Apr 26, 2017

Restarted, again.

@arneke
Copy link
Copy Markdown
Contributor Author

arneke commented May 5, 2017

I don't get why this build is so unpredictable on Travis.

I cleaned everything, removed all containers and all images before I ran "make" to test. And repeated after this failure, without seeing the issues.

@tomav Can you test locally ?

@tomav
Copy link
Copy Markdown
Contributor

tomav commented May 10, 2017

Worked locally. Really hard to understand Travis...

@tomav
Copy link
Copy Markdown
Contributor

tomav commented May 10, 2017

It worked!

@tomav tomav merged commit c053049 into docker-mailserver:master May 10, 2017
@tomav
Copy link
Copy Markdown
Contributor

tomav commented May 10, 2017

Thank you @arneke

RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
* First pass at adding support for pipe and filter in Sieve, excluding unit tests.
* Reduce insight into copied scripts
* Added test
* Adjustments related to switch to 16.04
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.

2 participants