<table><tr><td style="">drosca requested changes to this revision.<br />drosca added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12522">View Revision</a></tr></table><br /><div><div><p>This is wrong, it would accept <strong>all</strong> certificate errors triggered by one page, regardless of what error it is - it may even be error with completely different certificate, from different resource on page. If anything, this should only be offered to reject all certificate errors from one page, but even then it doesn't really make sense.</p>

<p>What should be done to fix that bug is to offer <tt style="background: #ebebeb; font-size: 13px;">NoForThisSession</tt> similar to <tt style="background: #ebebeb; font-size: 13px;">OnlyForThisSession</tt> with same logic. Event this logic is not really correct due to very limited QtWebEngine API here, but is good enough to be used for one browsing session (it only stores url and certificate error type).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R875 Falkon</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12522">https://phabricator.kde.org/D12522</a></div></div><br /><div><strong>To: </strong>anmolgautam, drosca, Falkon<br /><strong>Cc: </strong>falkon, herrold, anmolgautam, SGOrava, iodelay, spoorun, navarromorales, cochise, clivej, ach<br /></div>