Summary: | Flatpak KCM generates broken override config | ||
---|---|---|---|
Product: | [Applications] systemsettings | Reporter: | ratijas <me> |
Component: | kcm_flatpak | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | joshiesuhaas0, nate, putr4.s |
Priority: | NOR | ||
Version: | master | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/flatpak-kcm/commit/29e236f97becfd699e8a38fd4101d28428e4a3dc | Version Fixed In: | 5.27.5 |
Description
ratijas
2023-02-09 11:25:04 UTC
Keeping severity NOR since we've disabled the environment variable UI for now to avoid this; otherwise it would be HI or even VHI. (In reply to Nate Graham from comment #1) > Keeping severity NOR since we've disabled the environment variable UI for > now to avoid this; otherwise it would be HI or even VHI. Unfortunately that UI still appears if a Flatpak app has at least one environment variable override added by the user (for example via Flatseal), and while neither editing an existing environment variable's value nor adding a new one works (so no damage is done in those cases) disabling/deleting environment variables do take effect, and in that case it also generates an invalid override file and prevents the affected app from launching. > Unfortunately that UI still appears if a Flatpak app has at least one environment variable override added by the user (for example via Flatseal), and while neither editing an existing environment variable's value nor adding a new one works (so no damage is done in those cases) disabling/deleting environment variables do take effect, and in that case it also generates an invalid override file and prevents the affected app from launching.
Right… There were two functions: for loading default values, and for existing overrides. I ended up only commenting out default values.
Git commit ce407199aae085210107e57fef0fd8ed7b0b63bc by ivan tkachenko. Committed on 02/03/2023 at 22:10. Pushed by ratijas into branch 'master'. FlatpakPermission: Disable loading environment from user overrides as well M +6 -3 flatpakpermission.cpp https://invent.kde.org/plasma/flatpak-kcm/commit/ce407199aae085210107e57fef0fd8ed7b0b63bc Git commit 7e45e4931ea1a84000255c54ae62bf2d2b8bc2d2 by ivan tkachenko. Committed on 02/03/2023 at 22:17. Pushed by ratijas into branch 'Plasma/5.27'. FlatpakPermission: Disable loading environment from user overrides as well (cherry picked from commit ce407199aae085210107e57fef0fd8ed7b0b63bc) M +6 -3 flatpakpermission.cpp https://invent.kde.org/plasma/flatpak-kcm/commit/7e45e4931ea1a84000255c54ae62bf2d2b8bc2d2 A possibly relevant merge request was started @ https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/107 Git commit 12ab644f8a3d64a35f3a3608b24caee86d48b956 by ivan tkachenko. Committed on 18/04/2023 at 17:52. Pushed by ratijas into branch 'master'. FlatpakPermissionModel: Enable loading and adding environment variables Since we have reimplemented all the reading and writing code and added input validation to protect against empty keys (env. var. names), we can finally safely bring them back in the UI. M +0 -1 autotests/flatpakpermissiontest.cpp M +4 -9 flatpakpermission.cpp M +1 -1 package/contents/ui/permissions.qml https://invent.kde.org/plasma/flatpak-kcm/commit/12ab644f8a3d64a35f3a3608b24caee86d48b956 Git commit 29e236f97becfd699e8a38fd4101d28428e4a3dc by ivan tkachenko. Committed on 18/04/2023 at 20:15. Pushed by ratijas into branch 'Plasma/5.27'. FlatpakPermissionModel: Enable loading and adding environment variables Since we have reimplemented all the reading and writing code and added input validation to protect against empty keys (env. var. names), we can finally safely bring them back in the UI. (cherry picked from commit 12ab644f8a3d64a35f3a3608b24caee86d48b956) M +0 -1 autotests/flatpakpermissiontest.cpp M +4 -9 flatpakpermission.cpp M +1 -1 package/contents/ui/permissions.qml https://invent.kde.org/plasma/flatpak-kcm/commit/29e236f97becfd699e8a38fd4101d28428e4a3dc |