Summary: | Kmail Sends StartTls but shouldn't when connecting to sieve | ||
---|---|---|---|
Product: | [Unmaintained] kmail | Reporter: | DonMartio <donmartio> |
Component: | general | Assignee: | kdepim bugs <kdepim-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | donmartio |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | Version Fixed 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 |