Bug 473833

Summary:	kwalletd6 does not find wallet data
Product:	[Frameworks and Libraries] frameworks-kwallet	Reporter:	Nate Graham <nate>
Component:	general	Assignee:	Valentin Rusu <valir>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugseforuns, kdelibs-bugs-null, nicolas.fella
Priority:	NOR	Keywords:	qt6
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/kwallet/-/commit/859672094acca2ae5e9989590211f5a0f8408dd2	Version Fixed In:
Sentry Crash Report:

Description Nate Graham 2023-08-28 04:54:28 UTC

As of today in my Plasma 6 session (built from git master daily), the KDE Wallet I set up years ago is no longer being noticed by the kded6 version of the wallet module. For example whenever I connect to my home WiFi network, I'm prompted to set up a new wallet, rather than the password to the network being looked up in the wallet.

$ qdbus org.kde.kded6 /kded org.kde.kded6.loadedModules
plasmavault
desktopnotifier
freespacenotifier
appmenu
baloosearchmodule
kded_bolt
gtkconfig
smart
proxyscout
devicenotifications
smbwatcher
kded_plasma-welcome
ksysguard
plasma_accentcolor_service
keyboard
printmanager
kded_accounts
ktimezoned
remotenotifier
bluedevil
kscreen
networkmanagement
statusnotifierwatcher


It looks like the wallet module isn't running or isn't ported to Qt6 or something?

However, manually launching /usr/bin/kded5 in addition to my built-from-source kded6 does not resolve the issue.

Comment 1 Nicolas Fella 2023-08-28 08:54:16 UTC

Kwallet is not a kded module, it's entirely expected to not show up there

Comment 2 Nicolas Fella 2023-08-28 09:06:16 UTC

I have not looked closely, but I'd guess it's related to https://invent.kde.org/frameworks/kwallet/-/merge_requests/64

That said it's quite likely developer setup weirdness with DBus, not an actual bug

Comment 3 Nicolas Fella 2023-08-28 10:21:57 UTC

I know what's going on, and it is a proper bug

Comment 4 Bug Janitor Service 2023-08-28 10:28:27 UTC

A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kwallet/-/merge_requests/65

Comment 5 Nicolas Fella 2023-08-28 15:09:34 UTC

Git commit 859672094acca2ae5e9989590211f5a0f8408dd2 by Nicolas Fella.
Committed on 28/08/2023 at 12:26.
Pushed by nicolasfella into branch 'master'.

Properly fix wallet location

AppDataLocation contains the major version of kwalletd, which is undesired here

The code hacked around that, but that broken the moment we renamed from 5 to 6

Instead of working around AppDataLocation use GenericDataLocation and build the proper path ourselves

M  +1    -8    src/runtime/kwalletd/backend/kwalletbackend.cc

https://invent.kde.org/frameworks/kwallet/-/commit/859672094acca2ae5e9989590211f5a0f8408dd2