Review Request: Let kio_smb store passwords in KWallet again.

Commit Hook null at kde.org
Thu Oct 18 01:50:45 BST 2012


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


This review has been submitted with commit 9c0021e98c0c6a808ffa82449d355a64ecef4552 by Dawit Alemayehu to branch KDE/4.9.

- Commit Hook


On Oct. 16, 2012, 10:59 a.m., Gordian Klein wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106909/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2012, 10:59 a.m.)
> 
> 
> Review request for KDE Runtime.
> 
> 
> Description
> -------
> 
> Hi, this patch fixes an issue where kio_smb wouldnt save the credentials in kwallet wether one checks the "save password" box or not.
> 
> In slavebase.h is the following is stated:
> "Starting with KDE 4.7, calling openPasswordDialog will no longer store
> passwords automatically for you. This was done to avoid accidental storage
> of incorrect or invalid password information."
> 
> So i added a call to cacheAuthentication() to store the password permanently.
> 
> 
> Diffs
> -----
> 
>   kioslave/smb/kio_smb_auth.cpp 1010ffc 
> 
> Diff: http://git.reviewboard.kde.org/r/106909/diff/
> 
> 
> Testing
> -------
> 
> Used dolphin to access a samba share and checked the "save password" dialog. Verified that the password gets stored in KWallet.
> 
> 
> Thanks,
> 
> Gordian Klein
> 
>

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


More information about the kde-core-devel mailing list