D9875: Extend parsing ssh prompt
Martin Koller
noreply at phabricator.kde.org
Sat Jun 13 09:37:23 BST 2020
mkoller reopened this revision.
mkoller added a comment.
This revision is now accepted and ready to land.
This patch breaks usage for git (and probably others):
git first asks for a "Username for 'https:...." which leads to ksshaskpass open the input dialog but the typed-in user
is no longer stored into the wallet!!
(See case TypeClearText)
This leads to git again and again ask for the Username on each invokation.
Please ensure that even the Username is stored again _in the same folder_ in kwallet as before (e.g. below Passwords)
otherwise it also breaks using existing passwords. E.g. reading
if (type != TypePassword) {
QByteArray retrievedBytes;
wallet->readEntry(identifier, retrievedBytes);
is wrong (backwards incompatible).
REPOSITORY
R105 KDE SSH Password Dialog
REVISION DETAIL
https://phabricator.kde.org/D9875
To: fvogt, pali, lbeltrame
Cc: mkoller, lbeltrame, ngraham, fvogt, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200613/c5579d2d/attachment.htm>
More information about the Plasma-devel
mailing list