| Summary: | Kmail Sends StartTls but shouldn't when connecting to sieve | ||
|---|---|---|---|
| Product: | [Unmaintained] kmail | Reporter: | DonMartio <donmartio> |
| Component: | general | Assignee: | kdepim bugs <pim-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | donmartio |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
DonMartio
2009-11-03 19:03:26 UTC
If i call the server via telnet it gives me the capabilities in return.
In my case there is no "STARTTLS" on this special server in this response.
I've tested a little bit around and checked the sources of cyrus-2.3.16 and kdepimlibs-4.4.0.
As far as i understood it cyrus cuts the connection if a STARTTLS token is sent and TLS is not configured/supported.
In the kdeplimlibs sources i found that the kio_sieve slave ignores that there is no STARTTLS capability received from the server.
I changed the line 368 in kioslave/sieve/sieve.cpp
from
if (useTLSIfAvailable && QSslSocket::supportsSsl()) {
to
if (useTLSIfAvailable && m_supportsTLS && QSslSocket::supportsSsl()) {
and it's working again.
I'm not sure if this the right way to solve this. May someone can check it an put it into some future release so i don't have to keep my own kdepimlibs version.
Greetings so far.
SVN commit 1092916 by tmcguire: Don't send STARTTLS when it is not advertised, even if we requested it (but not if m_allowUnencrypted is set to false). Thanks to Don Martio <donmartio at larkos dot de> for providing the solution. BUG: 212951 M +2 -3 sieve.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1092916 SVN commit 1092921 by tmcguire: Backport r1092916 by tmcguire from trunk to the 4.4 branch: Don't send STARTTLS when it is not advertised, even if we requested it (but not if m_allowUnencrypted is set to false). Thanks to Don Martio <donmartio at larkos dot de> for providing the solution. CCBUG: 212951 M +2 -3 sieve.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1092921 |