Created attachment 147295 [details] error message SUMMARY Error message when clicking "Get new icons..." in system settings STEPS TO REPRODUCE 1.Go to System settings / Appearence / Icons 2. Click the "Get new icons..." button OBSERVED RESULT Error message (cf screenshot) EXPECTED RESULT Display available Icons from servers SOFTWARE/OS VERSIONS Linux/KDE Plasma: KDE Neon 5.24 (available in About System) KDE Plasma Version: 5.24.2 KDE Frameworks Version: 5.91 Qt Version: 5.15.3 ADDITIONAL INFORMATION
Can't confirm this behavior. Everything works great for me. SOFTWARE/OS VERSIONS Distro: Manjaro KDE Plasma Version: 5.24.2 KDE Frameworks Version: 5.91 Qt Version: 5.15.2
Happens also in Fedora 35. Maybe it is linked to french language ?
Are you able to try in another language please, that way we can confirm if that's the issue. I've also not been able to replicate this issue on Fedora 36 with "American English" Language and en_AU (Australia) keyboard layout.
My system is in french, but in System settings / Language, only American English is present... I don't know why and I don't know how to make the changes necessary to test the issue.
(In reply to makosol from comment #2) > Happens also in Fedora 35. Maybe it is linked to french language ? Can reproduce when setting the language to french. With my default en_US it works fine. In attica one gets a QNetworkReply::RemoteHostClosedError, @leinir any ideas?
i mean, apart from the remote host refusing to serve us seeming like it has to be a service side issue, i'm really not sure... Especially given how this code just hasn't changed on our side for ages, and it's suddenly broken... Maybe something with the new caching setup causes it to fail if there's a language set in the request header? (which we don't do ourselves, but i expect it gets sent along by qnam, not really sure). A quick test just trying to manually set the accept-language header to fr seems to just result in a correctly returned list of categories, though, so i'm really not sure what's going on here.
According to KNS, I found out an interesting thing: if the dialog opens in a category that is not translated (or has Latin characters), everything is displayed without problems. Those. sections konsole, yakuake, etc. open, but which contain non-English characters not. Tested with renaming the translation of /usr/share/knsrcfiles/*
This error was caused by this line in KF5 Attica code: https://invent.kde.org/frameworks/attica/-/blob/dde032d415e26a87f900eb6f0131f5114f0e72f9/src/provider.cpp#L1724 User-Agent header was set to UTF-8 value, "knewstuff-dialog/1.0 (+Вход в систему)" in my case. It's not valid according to HTTP standard (as I know), so HTTP server breaks connection. (See also: https://stackoverflow.com/questions/10384741/is-a-unicode-user-agent-legal-inside-an-http-header) Commenting this line solves issue for me.
Maybe it's worth to encode m_additionalAgentInformation with QUrl::toPercentEncoding(...) before setting it to header.
I think it is worth rethinking if we even want the user-visible name inside the user agent. Instead I would propose to use the knsrc file basename. This is an identifier we use when creating the engine.
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knewstuff/-/merge_requests/173
@Yaroslav Sidlovsky Thanks for the valuable investigation!
Thank you so much Yaroslav! Amazing work!
*** Bug 451087 has been marked as a duplicate of this bug. ***
Git commit 693a2ea3926400b1482888a2df2c532852b8f971 by Nate Graham, on behalf of Alexander Lohnau. Committed on 23/03/2022 at 22:58. Pushed by ngraham into branch 'master'. Do not set user-visible name as additional agent information Instead we want the filename, which is more useful as an identifier anyways, because we would otherwise have to check which user-visible names originate from which knsrc file. M +4 -1 src/core/engine.cpp https://invent.kde.org/frameworks/knewstuff/commit/693a2ea3926400b1482888a2df2c532852b8f971
Created attachment 147691 [details] attachment-12149-0.html No bugfix release? On Thu, Mar 24, 2022, 9:32 AM Nate Graham <bugzilla_noreply@kde.org> wrote: > https://bugs.kde.org/show_bug.cgi?id=451165 > > Nate Graham <nate@kde.org> changed: > > What |Removed |Added > > ---------------------------------------------------------------------------- > Version Fixed In| |5.93 > > -- > You are receiving this mail because: > You are on the CC list for the bug.
That would be up to distro packagers or the Frameworks folks. Frameworks doesn't get automatic bugfix releases, just the normal monthly releases. So someone would need to request one. Would you be interested in doing so?
Created attachment 147692 [details] attachment-12570-0.html If this patch cleanly applies to the current frameworks release I guess there's no need for a separate release. I've already proposed the use of the patch for Fedora KDE. On Thu, Mar 24, 2022, 10:13 AM Nate Graham <bugzilla_noreply@kde.org> wrote: > https://bugs.kde.org/show_bug.cgi?id=451165 > > --- Comment #17 from Nate Graham <nate@kde.org> --- > That would be up to distro packagers or the Frameworks folks. Frameworks > doesn't get automatic bugfix releases, just the normal monthly releases. So > someone would need to request one. Would you be interested in doing so? > > -- > You are receiving this mail because: > You are on the CC list for the bug.
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knewstuff/-/merge_requests/175
There is another place where I am not sure how relevant it even is. Once that is figured out I will reach out to distros and make a backport request.
Git commit 3ec15eaac29d6b73b078ffe1665fc68fc2beeeac by Alexander Lohnau. Committed on 24/03/2022 at 06:02. Pushed by alex into branch 'master'. Do not set user-visible name as additional agent information when copying provider Instead we want the filename, which is more useful as an identifier anyways, because we would otherwise have to check which user-visible names originate from which knsrc file. M +1 -1 src/core/engine.cpp https://invent.kde.org/frameworks/knewstuff/commit/3ec15eaac29d6b73b078ffe1665fc68fc2beeeac
*** Bug 451088 has been marked as a duplicate of this bug. ***
*** Bug 451611 has been marked as a duplicate of this bug. ***
*** Bug 451364 has been marked as a duplicate of this bug. ***
*** Bug 452157 has been marked as a duplicate of this bug. ***
*** Bug 453066 has been marked as a duplicate of this bug. ***
*** Bug 453954 has been marked as a duplicate of this bug. ***
*** Bug 457990 has been marked as a duplicate of this bug. ***
(In reply to Nate Graham from comment #22) > *** Bug 451088 has been marked as a duplicate of this bug. *** thank you all. Nice responsiveness ! Best regards
This bug report says that this is fixed in KDE Plasma Framework 5.93. I am running Ubuntu Studio 22.04.1 LTS with KDE Plasma 5.24.6 and Framework 5.95, but this error appears when I try to download a new "Login Screen (SDDM)" by clicking "Get New SDDM Themes..." (in "System Settings" under "Startup and Shutdown").
P.S. My language is en_US, but my locale is de_CH.
The locale doesn't matter; if your language is English, it's a different bug. Can you file a new bug report? Also please make sure to attach a screenshot of the error message. Thanks!
(In reply to Nate Graham from comment #32) > The locale doesn't matter; if your language is English, it's a different > bug. Can you file a new bug report? Also please make sure to attach a > screenshot of the error message. Thanks! Thanks, I started to file a new bug as you suggested, but can no longer reproduce the problem. Perhaps an update fixed it, or perhaps it was an issue with privileges or with the server connection. IAC, despite the thousands of icons available, I didn't find what I was looking for. I was hoping to find avatar images like the ones macOS and Windows provide fr their accounts and login screens.
the bug has been fixed now for 10 days during a KDE update. Thanks to the developers.