<?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>423424</bug_id>
          
          <creation_ts>2020-06-24 08:18:18 +0000</creation_ts>
          <short_desc>Kmail &quot;forces&quot; the user to accept invalid TLS certificates.</short_desc>
          <delta_ts>2021-11-16 17:06:40 +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>REOPENED</bug_status>
          <resolution></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>sknauss</cc>
          
          <cf_commitlink>https://invent.kde.org/pim/ksmtp/commit/fca378d55e223944ce512c9a8f8b789d1d3abcde</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>20</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1939462</commentid>
    <comment_count>0</comment_count>
    <who name="Damian Poddebniak">93s4m32gd2ab8ax6</who>
    <bug_when>2020-06-24 08:18:18 +0000</bug_when>
    <thetext>When the IMAP TLS certificate is bad, i.e. self-signed, kmail shows a warning with three buttons: &quot;Details&quot;, &quot;Continue&quot; and &quot;Cancel&quot;. When the user clicks on &quot;Cancel&quot;, kmail repeats the login process and shows the warning again immediately. This process continues in a loop, which can not be canceled by the user when clicking on &quot;Cancel&quot; (the only secure option).

The only way to &quot;escape&quot; from this loop is to click on &quot;Continue.&quot;, which might reveal the username and password.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1939465</commentid>
    <comment_count>1</comment_count>
    <who name="Damian Poddebniak">93s4m32gd2ab8ax6</who>
    <bug_when>2020-06-24 08:29:43 +0000</bug_when>
    <thetext>This also applies in a limited form (dialogs show up slower) for SMTP.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2060253</commentid>
    <comment_count>2</comment_count>
    <who name="Sandro Knauß">sknauss</who>
    <bug_when>2021-09-10 19:27:12 +0000</bug_when>
    <thetext>The vulnerable is now published under https://nostarttls.secvuln.info/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064695</commentid>
    <comment_count>3</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2021-09-28 15:59:56 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/pim/kimap/-/merge_requests/9</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064700</commentid>
    <comment_count>4</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2021-09-28 16:06:15 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/pim/kdepim-runtime/-/merge_requests/48</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064760</commentid>
    <comment_count>5</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-09-28 19:27:34 +0000</bug_when>
    <thetext>Git commit 7ee241898bc225237b3475f6c109ffc55a4a74c0 by Volker Krause.
Committed on 28/09/2021 at 15:58.
Pushed by knauss into branch &apos;release/21.08&apos;.

Disconnect rather than reconnect when not ignoring SSL errors

Reconnecting makes no sense, we&apos;ll just end up with the SSL error dialog
again and again in that case.

Not enough to fix 423424 by itself, but a necessary prerequisite.

M  +1    -4    src/sessionthread.cpp

https://invent.kde.org/pim/kimap/commit/7ee241898bc225237b3475f6c109ffc55a4a74c0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064770</commentid>
    <comment_count>6</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-09-28 19:58:18 +0000</bug_when>
    <thetext>Git commit edb7f6fdea2c9f44085a042531f56223f3fd8a2f by Volker Krause.
Committed on 28/09/2021 at 16:05.
Pushed by knauss into branch &apos;release/21.08&apos;.

Consider the online state when attempting to reconnect

There&apos;s actually a comprehensive error condition handling in the method
above which properly distinguishing between transient and persistent
problems, but we just ignore that decision here and continuously
reconnect.

Together with https://invent.kde.org/pim/kimap/-/merge_requests/9 this
fixes the infinite SSL error dialog loop when rejecting to ignore an SSL
error to a large extend. You still get the dialog twice now, and then
after a few minutes again as this is considered to be a transient error
(e.g. caused by capture portals). This at least gives you the opportunity
now to actually fix the configuration or remove the resource.

(Bug 423424 remains open for SMTP)

M  +1    -1    resources/imap/imapresourcebase.cpp

https://invent.kde.org/pim/kdepim-runtime/commit/edb7f6fdea2c9f44085a042531f56223f3fd8a2f</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064966</commentid>
    <comment_count>7</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2021-09-29 15:42:47 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/pim/ksmtp/-/merge_requests/10</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064985</commentid>
    <comment_count>8</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-09-29 17:25:41 +0000</bug_when>
    <thetext>Git commit fca378d55e223944ce512c9a8f8b789d1d3abcde by Volker Krause.
Committed on 29/09/2021 at 15:41.
Pushed by knauss into branch &apos;release/21.08&apos;.

Emit an error rather than reconnect when SSL errors are not ignored

Not ignoring SSL certificate errors now results in a delivery error rather
than a loop on the SSL error dialog.

M  +5    -4    src/sessionthread.cpp

https://invent.kde.org/pim/ksmtp/commit/fca378d55e223944ce512c9a8f8b789d1d3abcde</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2077048</commentid>
    <comment_count>9</comment_count>
    <who name="Sandro Knauß">sknauss</who>
    <bug_when>2021-11-12 12:21:13 +0000</bug_when>
    <thetext>This was rechecked from the NO STARTTLS team with the current version 5.18.40 and this bug is not completly fixed:

&quot;The certificate loop for IMAP in the account wizard is also still present, kmail keeps reconnecting for me. I cannot even accept the invalid certificate, because kmail continues reconnecting and showing dialogs. I can provide a screen recording if needed. For SMTP it seems to be fixed (the dialog only appears once).&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2077768</commentid>
    <comment_count>10</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2021-11-15 17:20:35 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/pim/kimap/-/merge_requests/10</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2078080</commentid>
    <comment_count>11</comment_count>
    <who name="Volker Krause">vkrause</who>
    <bug_when>2021-11-16 17:06:40 +0000</bug_when>
    <thetext>Git commit cbd3a03bc1d2cec48bb97570633940bbf94c34fa by Volker Krause.
Committed on 15/11/2021 at 17:18.
Pushed by knauss into branch &apos;release/21.12&apos;.

Treat SSL handshake errors as fatal also when using STARTTLS

This fixes the infinite SSL error dialog loop also when using
STARTTLS, the previous fix was only effective for direct TLS
connections.

M  +9    -13   src/loginjob.cpp

https://invent.kde.org/pim/kimap/commit/cbd3a03bc1d2cec48bb97570633940bbf94c34fa</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>