Review Request 111335: Fix for one of the oldest KIO bugs: multiple dialogs when KIO encounters SSL errors

[Dawit Alemayehu]

> On July 13, 2013, 6:32 a.m., David Faure wrote:
> > Almost there, thanks.
> > 
> > I'm surprised by the dispatch (to call one of the many uidelegate methods), followed by an un-dispatch (what's the term?), i.e. these many methods all end up calling the same method anyway.... I can see how it makes the uidelegate api look nicer, but this is all internal, nobody else than the slave should end up calling these methods (if apps need messageboxes they can just use KMessageBox directly). I'm not vetoing this solution, maybe you can leave it, I'm just wondering what's the purpose.

Well to be honest I just simply hate marking publicly declared APIs as internal. That is why I did what I did. In hindsight though I should resisted my own dislike and and simply created the public API and marked it internal. You are right. The rolling and unrolling (making up words here) of the dispatch call make little sense, especially when it is done to avoid ones own pet peeve.


- Dawit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111335/#review35907
-----------------------------------------------------------


On July 13, 2013, 1:30 a.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111335/
> -----------------------------------------------------------
> 
> (Updated July 13, 2013, 1:30 a.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> The attached patch addresses one of the oldest bugs in KIO. Due to the muti-process nature of KIO, if any of the ioslaves encounter something that requires user input, the user might end up getting prompted multiple times. The best example of this is SSL error warnings sent to the client by kio_http. The patch completely resolves this problem using the same approach as kpasswdserver, but without the need for an additional kded process.
> 
> 
> This addresses bugs 154100 and 265228.
>     http://bugs.kde.org/show_bug.cgi?id=154100
>     http://bugs.kde.org/show_bug.cgi?id=265228
> 
> 
> Diffs
> -----
> 
>   kio/CMakeLists.txt 4854829 
>   kio/kio/job_p.h 0c1fd64 
>   kio/kio/jobuidelegate.h 25e0728 
>   kio/kio/jobuidelegate.cpp 85679c2 
>   kio/kio/scheduler.cpp 802f8b8 
>   kio/kio/slaveinterface.h 4bfcec8 
>   kio/kio/slaveinterface.cpp aa0fc44 
>   kio/kio/slaveinterface_p.h e31ec5e 
>   kio/kio/usernotificationhandler.cpp PRE-CREATION 
>   kio/kio/usernotificationhandler_p.h PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111335/diff/
> 
> 
> Testing
> -------
> 
> Visit a site that throws up SSL warnings and causes KIO to show more than one error dialog.
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130713/b1ed627d/attachment.htm>