http://bugzilla.opensuse.org/show_bug.cgi?id=1167985 http://bugzilla.opensuse.org/show_bug.cgi?id=1167985#c2 Wolfgang Bauer changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wbauer@tmo.at --- Comment #2 from Wolfgang Bauer --- (In reply to Dirk Weber from comment #0) With Leap 15.2 Beta a bug similar to bug 1049975 is back. That's likely an upstream regression then I suppose. I don't see a change to the code that was fixed back then though... -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1167985 http://bugzilla.opensuse.org/show_bug.cgi?id=1167985#c4 --- Comment #4 from Wolfgang Bauer --- (In reply to Wolfgang Bauer from comment #3)

It seems to work fine here though. Ah, wait. The setting is stored in the config file, but konqueror (or rather kded) does ask again after I restart it....

-- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1167985 http://bugzilla.opensuse.org/show_bug.cgi?id=1167985#c6 Wolfgang Bauer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CONFIRMED --- Comment #6 from Wolfgang Bauer --- I think I know what happens: kcookiejar itself has similar code to read/write the settings string as the settings module. Previously, only the KCM was changed to handle the string with spaces (by removing the spaces), but not kcookiejar itself, i.e. the daemon itself still doesn't recognize the setting "Accept For Session" (with spaces). Now I just wonder how the previous fix ever worked at all though... :-/ I created a "proof of concept" patch to fix it, by making kcookiejar strip the spaces too, like the old fix did for the KCM. Packages are available here: https://download.opensuse.org/repositories/home:/wolfi323:/branches:/openSUS... Please try them and report back if it works then with "Accept For Session" in the config file. (you should reboot after installing the packages to make sure that the old daemon is not running anymore) -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1167985 http://bugzilla.opensuse.org/show_bug.cgi?id=1167985#c8 Wolfgang Bauer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |IN_PROGRESS --- Comment #8 from Wolfgang Bauer --- (In reply to Dirk Weber from comment #7)

I found out that on all my productive systems the ~/.config/kcookiejarrc is readonly with timestamps from 2017. Ok, so I'll better check whether Leap 15.1 is also affected (I'm not aware of any bug reports though).

It seems to store ini values with blanks is an error prone approach. Having spaces in the value is not a problem per se, although it actually was never intended here.

What happened was that a translator changed the string in the settings module to contain spaces years ago (2014, during the port to Qt5), without noticing that it is actually written to the config file as well, which broke reading this setting (as the code that compares the value was not changed). https://cgit.kde.org/kio.git/commit/src/kcms/kio/kcookiespolicyselectiondlg.... And the previous fix obviously was not complete. (I'm pretty sure I did test it back then, but that's been a while) Just removing the spaces again (i.e. reverting the original change that caused the problems) would probably not be a good idea either, as the config file may contain it with spaces already.

I tested your packages. I installed them and rebooted. The setting is stored in the kcookiejarrc with blanks. After starting konqueror again it recognises the setting correctly.

After rebooting again it still works correctly.

I would say your correction is working.

Great, thanks! Good to hear. I'll propose the fix upstream first though, but I'll make sure it will land in Leap 15.2.

-- You are receiving this mail because: You are on the CC list for the bug.