UshakovVasilii / gnome-shell-extension-freon

Shows CPU temperature, disk temperature, video card temperature (NVIDIA/Catalyst/Bumblebee&NVIDIA), voltage and fan RPM
https://extensions.gnome.org/extension/841/freon
GNU General Public License v2.0
431 stars 77 forks source link

fix pkexec support and do not call pkexec at all if the install dir is not found #274

Closed illwieckz closed 10 months ago

illwieckz commented 10 months ago

fix pkexec support and do not call pkexec at all if the install dir is not found

The combination of the presence of that “broken lookup” bug and the lack of that “not found” test should have been wrong but not that bad: it should have just let pkexec ask the user to install a file that doesn't exist, then fail and reset the option and not annoy the user more than simply failing to use the option.

BUT GNOME Shell has a very nasty bug where sometime all new windows are opened in background, and when the bug occurs, this also affects the pkexec window, meaning the desktop freezes waiting for the user to input the password, but without displaying the password prompt to the user. This is not a bug in this extension, this is a bug in GNOME Shell, and GNOME Shell people requiring to use pkexec instead of non-interactive sudo while the said bug exists is just making them put the user at risk of locking the user desktop without any way to unlock it (unless you open a tty or connect from another computer through ssh, and kill pkexec).

After this patch is merged, if GNOME shell locks the user desktop because of the said bug, it will be 100% GNOME shell developer's fault, not ours, as it should be.