https://bugs.kde.org/show_bug.cgi?id=458318
Bug ID: 458318
Summary: Deprecate `org.kde.kwalletd5` API
Product: frameworks-kwallet
Version: 5.97.0
Platform: unspecified
OS: Linux
Status: REPORTED
Severity: wishlist
Priority: NOR
Component: general
Assignee: va...@kde.org
Reporter: mk.mat...@gmail.com
CC: kdelibs-b...@kde.org
Target Milestone: ---
SUMMARY
Now that https://invent.kde.org/frameworks/kwallet/-/merge_requests/11 has been
merged, and KWallet Framework supports the Secret Service API
(`org.freedesktop.secrets`) as of 5.97.0, can the entire `org.kde.kwalletd5`
API finally be marked as deprecated? This should encourage more KDE apps to
migrate to QtKeyChain, which supports both API backends.
SOFTWARE/OS VERSIONS
KDE Frameworks Version: 5.98.0+
ADDITIONAL DETAILS
`org.freedesktop.secrets` is the new standard API supported by GNOME keyring,
KeePassXC, and others. See discussion in
https://bugs.kde.org/show_bug.cgi?id=313216 and its duplicates.
Early attempts to replace KWallet with `org.freedesktop.secrets` were abandoned
due to lack of developer time. This has apparently plagued KWallet since those
times, as evidenced also by the difficulty finding reviewers for the
above-mentioned MR.
The situation was discussed back in the day on the Plasma-devel mailing list -
https://mail.kde.org/pipermail/plasma-devel/2016-July/055641.html - and the
consensus seemed to be to migrate the KDE apps to QtKeyChain, which would
separate the API from the backend implementation. QtKeyChain added support for
`org.freedesktop.secrets` (via `libsecret`) a few months later:
https://github.com/frankosterfeld/qtkeychain/pull/62 .
However, there was little incentive for KDE apps to make this migration while
the old `org.kde.kwalletd5` API is still around, and it was not possible to
remove it while apps are still using it. Now that KWallet supports the
`org.freedesktop.secrets` API, the old API can finally be deprecated to
encourage the apps to migrate. Then, maybe, it can be dropped completely in
KF6.
There is still one migration concern as discussed in
https://invent.kde.org/frameworks/kwallet/-/merge_requests/11#note_428272 and
its replies, but that should probably be handled by QtKeyChain. I will post on
that to their github issues.
--
You are receiving this mail because:
You are watching all bug changes.
