interface/screen_inhibit_control: Improve screen inhibit control for use on core #14134
Conversation
er-vin
commented
Jun 26, 2024
er-vin
commented
- Make it available on Ubuntu Core
- Receive notifications for inhibit changes, plugged application could change the inhibit status but had no way to be notified when it actually changes.
ernestl
added
the
Needs security review
bboozzoo
changed the title
er-vin
force-pushed
the
improve-screen-inhibit-control-for-use-on-core
branch
from
bd22eba
to
b6b55ef
Compare
er-vin
force-pushed
the
improve-screen-inhibit-control-for-use-on-core
branch
from
b6b55ef
to
88e6436
Compare
ernestl
removed
the
Needs security review
There was a problem hiding this comment.
The addition of access to the HasInhibitChanged signal looks fine (it is included in this copy of the spec). Making the interface implicit on core seems suspect.
Checking my KDE Core Desktop VM, org.freedesktop.PowerManagement on the session bus is owned by org_kde_powerdevil running with the AppArmor label snap.plasma-desktop-session.plasma-powerdevil. Similarly org.freedesktop.ScreenSaver is owned by a process with the label snap.plasma-desktop-session.plasma-kwin-wayland. So allowing communication with label=unconfined does not sound like it would actually enable any communication here.
For proper operation on Core, the interface would need to be modified to allow a snap to provide the slot.
|
Changed milestone to 2.67, needs more time. |
This can be provided by a session snap in such a case
Plugged application could change the inhibit status but had no way to be notified when it actually changes.
er-vin
force-pushed
the
improve-screen-inhibit-control-for-use-on-core
branch
from
88e6436
to
98d006b
Compare
It has been reworked in this direction. |
