Bug 446925 - drkonqi does not persist authentication credentials even if save option is checked.
Summary: drkonqi does not persist authentication credentials even if save option is ch...
Status: REOPENED
Alias: None
Product: drkonqi
Classification: Applications
Component: general (show other bugs)
Version: 5.23.4
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 479705 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-12-13 11:57 UTC by Fabio C. Barrionuevo
Modified: 2024-03-12 18:06 UTC (History)
4 users (show)

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


Attachments
drkonqi kwallet (31.24 KB, image/png)
2021-12-13 11:57 UTC, Fabio C. Barrionuevo
Details
kwallet settings (60.99 KB, image/png)
2021-12-16 15:18 UTC, Fabio C. Barrionuevo
Details
application connected to kwallet (33.22 KB, image/png)
2021-12-16 15:19 UTC, Fabio C. Barrionuevo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fabio C. Barrionuevo 2021-12-13 11:57:04 UTC
Created attachment 144509 [details]
drkonqi kwallet

SUMMARY

When an error occurs, KDE Neon opens drkonqi to warn of the error. When trying to report an error, you are prompted for a login and password and there is an option to save this authentication credential.

The problem is that even checking the option to save the credentials if a new error happens, I will be forced to enter the authentication credentials again, which seems to me to be the wrong behavior for the option to save authentication credentials.

OBSERVED RESULT
Authentication information is always prompted when trying to report an error, even if the option to save authentication credentials was previously checked.

EXPECTED RESULT

Enter the authentication information, check the option to save this credential, and not have to re-enter this information at any time in the future, ie during the current session or after one or more cycles of turning KDE Neon on and off

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.23
KDE Plasma Version: 5.23.4
KDE Frameworks Version: 5.88.0
Qt Version: 5.15.3
Kernel Version: 5.11.0-41-generic (64-bit)
Graphics Platform: X11
Processors: 24 × AMD Ryzen 9 3900X 12-Core Processor
Memory: 31.3 GiB of RAM
Graphics Processor: GeForce GTX 1650/PCIe/SSE2

ADDITIONAL INFORMATION:

I checked and kwallet is active.
I don't know if this has anything to do with the bug to do with the problem, but there is an entry for drkonqi, and the values saved in it are from the attached screenshot.
I would enter the correct credentials again into drkonqi but apparently, this record is not updated in kwallet and no new deliveries are entered into kwallet.
Comment 1 Fabio C. Barrionuevo 2021-12-13 14:10:52 UTC
>> ADDITIONAL INFORMATION:
>> I checked and kwallet is active.
>> I don't know if this has anything to do with the bug to do with the problem, but there is an entry for drkonqi, and the values saved in it are >> from the attached screenshot.
>> I would enter the correct credentials again into drkonqi but apparently, this record is not updated in kwallet and no new deliveries are >> entered into kwallet.


ignore this part of the original report. I kwallet contains the correct credentials but drkonqi doesn't seem to be using them.
Comment 2 Harald Sitter 2021-12-14 08:20:27 UTC
Does your password contain any special characters other than:

%
&
>
,

I cannot reproduce the issue, new login data is correctly stored in the wallet and (incorrect) existing data is correctly updated. Not quite sure what to do here.
Comment 3 Harald Sitter 2021-12-14 08:22:08 UTC
Are you quite certain kwallet is actually enabled? In kwallet manager check settings->configure it should say the wallet subsystem is enabled at the very top.
Comment 4 Fabio C. Barrionuevo 2021-12-16 15:18:24 UTC
Created attachment 144605 [details]
kwallet settings

my kwallet settings
Comment 5 Fabio C. Barrionuevo 2021-12-16 15:19:10 UTC
Created attachment 144606 [details]
application connected to kwallet

the application connected to kwallet
Comment 6 Fabio C. Barrionuevo 2021-12-16 15:20:20 UTC
(In reply to Harald Sitter from comment #2)
> Does your password contain any special characters other than:
> 
> %
> &
> >
> ,
> 
> I cannot reproduce the issue, new login data is correctly stored in the
> wallet and (incorrect) existing data is correctly updated. Not quite sure
> what to do here.

my password only contains lowercase ASCII characters and numbers
Comment 7 Harald Sitter 2021-12-17 10:05:08 UTC
Git commit 89cc53e42f15f04de133fe2462814452a5e8ab98 by Harald Sitter.
Committed on 17/12/2021 at 10:00.
Pushed by sitter into branch 'master'.

add some debugging for wallet interaction

something is wonky there and it's hard to say why because we have zero
logging. add some.

this will be available in 5.24 and should help us debug why things
aren't going the way they should

M  +8    -0    src/bugzillaintegration/reportassistantpages_bugzilla.cpp

https://invent.kde.org/plasma/drkonqi/commit/89cc53e42f15f04de133fe2462814452a5e8ab98
Comment 8 Harald Sitter 2022-02-21 08:01:33 UTC
I think I found the cause, if the pertinent kwallet folder doesn't exist it will write them to the wrong folder and consequently fail to read (from the right folder).
Comment 9 Bug Janitor Service 2022-02-22 12:29:54 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/drkonqi/-/merge_requests/54
Comment 10 Nate Graham 2022-02-22 16:00:13 UTC
Git commit 79df7ec54bc6380d086cad0f837b39a105e0fe32 by Nate Graham, on behalf of Harald Sitter.
Committed on 22/02/2022 at 15:59.
Pushed by ngraham into branch 'master'.

make sure to create the wallet folder before using it

otherwise the default "" folder is used and we'll get confused
contexts where the write would write into "" but we'd read from
formdata. simply always ensure the folder gets created first

M  +5    -1    src/bugzillaintegration/reportassistantpages_bugzilla.cpp

https://invent.kde.org/plasma/drkonqi/commit/79df7ec54bc6380d086cad0f837b39a105e0fe32
Comment 11 Nate Graham 2022-02-22 16:01:23 UTC
Git commit c98f842926bd9c24f2f0e08988a36a0af2c6906a by Nate Graham, on behalf of Harald Sitter.
Committed on 22/02/2022 at 16:01.
Pushed by ngraham into branch 'Plasma/5.24'.

make sure to create the wallet folder before using it

otherwise the default "" folder is used and we'll get confused
contexts where the write would write into "" but we'd read from
formdata. simply always ensure the folder gets created first


(cherry picked from commit 79df7ec54bc6380d086cad0f837b39a105e0fe32)

M  +5    -1    src/bugzillaintegration/reportassistantpages_bugzilla.cpp

https://invent.kde.org/plasma/drkonqi/commit/c98f842926bd9c24f2f0e08988a36a0af2c6906a
Comment 12 Riccardo Robecchi 2022-12-28 08:00:11 UTC
Reopening as the bug is still present as of version 5.26.4.
Comment 13 Patrick Silva 2024-01-12 22:40:34 UTC
*** Bug 479705 has been marked as a duplicate of this bug. ***
Comment 14 rjx_ 2024-03-12 18:06:23 UTC
Can confirm that this is still an issue in drkonqi 6.0.1.
Makes bug reporting slightly less streamlined and easy, imo - would be great if this got fixed for ease of bug reporting.