Review Request 127121: KWallet::openWallet(Synchronous) : time out after 1 hour rather than 25 seconds.

David Faure faure at kde.org
Sat Feb 20 20:48:09 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127121/
-----------------------------------------------------------

(Updated Feb. 20, 2016, 8:48 p.m.)


Review request for KDE Frameworks and Valentin Rusu.


Changes
-------

Make it 2<<30 ms, i.e. 12 days if I counted correctly. I don't dare making it bigger, but it should be plenty ;)


Repository: kwallet


Description
-------

The default DBus timeout is 25 seconds, which means that if the user went to
get a cup of tea during session startup, when they come back they get prompted
with all sorts of additional non-kwallet password requests due to all kwallet
requests having timed out.

I added setTimeout in QDBusAbstractInterface in Qt 4.8 for things like this.

Testcase:
 eval `dbus-launch`
 export SSH_ASKPASS=$KDEDIR/bin/ksshaskpass
 ssh-add < /dev/null
(but the same happens with the IMAP resource etc.)


Diffs (updated)
-----

  src/api/KWallet/kwallet.cpp b72edad19840943f70755c8668668a1881f1fb39 

Diff: https://git.reviewboard.kde.org/r/127121/diff/


Testing
-------

(see commit log)


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160220/a59fa346/attachment.html>


More information about the Kde-frameworks-devel mailing list