-
Notifications
You must be signed in to change notification settings - Fork 143
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
Added ssl/tls support for sieve #701
Conversation
d499d26
to
aa75de9
Compare
96ebcbe
to
a997a9a
Compare
a997a9a
to
d2d99d6
Compare
f39e46b
to
4e93dc2
Compare
95a4393
to
02cd863
Compare
I think you need to update Hm_Sieve_Client_Factory and also the Tiki integration which uses a port of Hm_Sieve_Client_Factory to use the new parsing method. |
02cd863
to
cdccd59
Compare
Hello @kroky I made changes. Thanks. |
I don't see changes in php-sieve-manager library - how did we fix the tls parameter passing to client->connect that has no effect? |
cdccd59
to
9ee3811
Compare
@kroky @josaphatim Is this suitable to backport to branch 1.4.x, or we just wait for 1.5.0? |
It should be fine to be backported. |
Adding tls:// prefix to sieve configuration host will enable explicit STARTTLS
Adding ssl:// prefix to sieve configuration host will enable implicit SSL