Bug 70425 - KNode doesn't save passwords in KWallet correctly
Summary: KNode doesn't save passwords in KWallet correctly
Status: RESOLVED FIXED
Alias: None
Product: knode
Classification: Unmaintained
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-12-14 19:20 UTC by P. Varet
Modified: 2003-12-17 23:03 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description P. Varet 2003-12-14 19:20:17 UTC
Version:            (using KDE KDE 3.1.94)
Installed from:    Gentoo Packages
Compiler:          gcc (GCC) 3.2.3 20030422 (Gentoo Linux 1.4 3.2.3-r3, propolice) 
OS:          Linux

SUMMARY:

On news servers where a login/password is needed, if the KWallet service is used, KNode always 'forgets' the password after you've closed it. (If KWallet is not used, then KNode displays a warning pop-up when it starts, but otherwise recalls passwords correctly.)

(Note, this is not the same bug as 66336. In fact, the patch suggested in 66336 already seems to have been applied in this version.)


HOW TO REPRODUCE:

0) Make sure KWallet is on.
1) Start KNode.
2) Add new server with a login/pass.
3) Save settings.
4) If the default wallet isn't open, a KWallet password dialog pops up.
5) Open the KWallet manager. You can check that your KNode password is there alright.
6) Close KNode.
7) This causes the KWallet auth dialog to pop up -again- (???). Give it the wallet's password or authorize KNode to access it, depending on the case.
8) Restart KNode. Go to the server you've just added.
9) BUG: KNode has forgotten the password and asks for it again.


ADDITIONAL NOTES:

Apparently, if you go peek at your wallet after KNode has closed, your password *IS* in there. It is when KNode restarts that the 'forgetting' seems to take place. Odd.


SUGGESTED TEMPORARY WORKAROUND:

Either re-enter the news server password everytime, or turn the KWallet service off, if you don't mind the warning popup KNode displays when it starts.
Comment 1 Volker Krause 2003-12-15 17:06:53 UTC
Exactly the same here (CVS Head from 2003-12-15).
The password is stored correctly in the wallet, but knode doesn't use it. Knode seems to only access the wallet when being closed (judging from kwallets password dialog).
Comment 2 Thomas Vollmer 2003-12-17 13:39:51 UTC
Same for me. CVS HEAD from 16.12.2003.

First thought was, it is bug is caused because I use it from kontact.

Thomas
Comment 3 Volker Krause 2003-12-17 23:03:06 UTC
Subject: kdelibs/kwallet/client

CVS commit by vkrause: 

Fix a typo in folderDoesNotExist() causing it to always return true.

Fixes #70425.

CCMAIL: 70425-done@bugs.kde.org


  M +1 -1      kwallet.cc   1.41


--- kdelibs/kwallet/client/kwallet.cc  #1.40:1.41
@@ -618,5 +618,5 @@ void Wallet::walletOpenResult(int id) {
 
 bool Wallet::folderDoesNotExist(const QString& wallet, const QString& folder) {
-DCOPReply r = DCOPRef("kded", "kwalletd").call("fodlerDoesNotExist", wallet, folder);
+DCOPReply r = DCOPRef("kded", "kwalletd").call("folderDoesNotExist", wallet, folder);
 bool rc = true;
         if (r.isValid()) {