<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>423423</bug_id>
          
          <creation_ts>2020-06-24 08:07:22 +0000</creation_ts>
          <short_desc>STARTTLS is ignored when &quot;Server requires authentication&quot; not checked in UI</short_desc>
          <delta_ts>2021-09-23 19:39:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>kmail2</product>
          <component>general</component>
          <version>5.13.3</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Damian Poddebniak">93s4m32gd2ab8ax6</reporter>
          <assigned_to name="kdepim bugs">pim-bugs-null</assigned_to>
          <cc>montel</cc>
    
    <cc>rdieter</cc>
    
    <cc>sknauss</cc>
          
          <cf_commitlink>https://invent.kde.org/pim/ksmtp/commit/60f73c69758fe40a027a8e7402127d085f18545a</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1939459</commentid>
    <comment_count>0</comment_count>
    <who name="Damian Poddebniak">93s4m32gd2ab8ax6</who>
    <bug_when>2020-06-24 08:07:22 +0000</bug_when>
    <thetext>The STARTTLS option of SMTP is ignored, when &quot;Server requires authentication&quot; is not checked. In this case kmail will send any mail in cleartext.

Tested with kmail2 5.13.3 (19.12.3).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2050389</commentid>
    <comment_count>1</comment_count>
    <who name="Damian Poddebniak">93s4m32gd2ab8ax6</who>
    <bug_when>2021-08-02 14:22:28 +0000</bug_when>
    <thetext>May I ask for an update? To be clear: we think that this is a securtiy vulnerability.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2050401</commentid>
    <comment_count>2</comment_count>
    <who name="Laurent Montel">montel</who>
    <bug_when>2021-08-02 14:46:06 +0000</bug_when>
    <thetext>(In reply to Damian Poddebniak from comment #1)
&gt; May I ask for an update? To be clear: we think that this is a securtiy
&gt; vulnerability.


&quot;We&quot; ? who is &quot;we&quot; ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2050405</commentid>
    <comment_count>3</comment_count>
    <who name="Damian Poddebniak">93s4m32gd2ab8ax6</who>
    <bug_when>2021-08-02 14:50:35 +0000</bug_when>
    <thetext>Ah sorry :-) I wrote that comment without thinking too much. We (me and some colleagues) performed a STARTTLS test some months ago, reported multiple vulnerabilities and are now in the process to consolidate the still open bugs.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2060249</commentid>
    <comment_count>4</comment_count>
    <who name="Sandro Knauß">sknauss</who>
    <bug_when>2021-09-10 19:22:37 +0000</bug_when>
    <thetext>The vulnerable is now published under https://nostarttls.secvuln.info/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2062788</commentid>
    <comment_count>5</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-09-21 17:26:49 +0000</bug_when>
    <thetext>Git commit 38a4c09427f3fdc04f9893f8eda3f6807d9a3203 by Volker Krause.
Committed on 21/09/2021 at 16:18.
Pushed by knauss into branch &apos;master&apos;.

Move establishing the TLS connection to Session

This means we now also enable TLS when not having a LoginJob, ie. on
servers not requiring authentication.

Doing the same for STARTTLS is the next step then.

M  +0    -2    src/loginjob.cpp
M  +1    -11   src/session.cpp
M  +11   -2    src/sessionthread.cpp
M  +2    -0    src/sessionthread_p.h

https://invent.kde.org/pim/ksmtp/commit/38a4c09427f3fdc04f9893f8eda3f6807d9a3203</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2063137</commentid>
    <comment_count>6</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2021-09-22 15:31:08 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/pim/ksmtp/-/merge_requests/8</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2063484</commentid>
    <comment_count>7</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-09-23 19:39:38 +0000</bug_when>
    <thetext>Git commit 60f73c69758fe40a027a8e7402127d085f18545a by Volker Krause.
Committed on 23/09/2021 at 16:02.
Pushed by knauss into branch &apos;master&apos;.

Move STARTTLS setup from LoginJob to Session

This is now done immediately after opening the connection, independent
of whether there is a LoginJob at all.

M  +5    -28   src/loginjob.cpp
M  +15   -2    src/session.cpp
M  +1    -0    src/session_p.h

https://invent.kde.org/pim/ksmtp/commit/60f73c69758fe40a027a8e7402127d085f18545a</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>