Consider supporting application indicators

Bug #591317 reported by Jorge Castro
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Application Indicators
Invalid
Undecided
Unassigned
Screenkey
Fix Released
Medium
Pablo SEMINARIO

Bug Description

Please consider supporting Ubuntu's application indicators!
https://wiki.ubuntu.com/DesktopExperienceTeam/ApplicationIndicators

Andrew (and471)
description: updated
Revision history for this message
Andrew (and471) wrote :

It already has support, however the problem is to do with how it is run.

If /dev/input/event0 has read access it runs and works fine, however if not, it has to be run with sudo.

When it is run with sudo, for some reason, appindicator hooks it into gtk.StatusIcon instead

This is a bug with appindicator and screenkey (only the necessary parts need be run was root)

Revision history for this message
Ted Gould (ted) wrote :

My guess would be that it's not finding the session bus. It seems to me that you really don't want all of screenkeys to run as root anyway. Would it make sense to have a part of it be a system dbus service, and then keep the UI part all in the user session? It seems like a smaller amount of code for a potential exploit.

Revision history for this message
Pablo SEMINARIO (pabluk) wrote :

Thanks Jorge and Andrew,
Could you please give me more information about your plataform? as GNU/Linux distro or python-appindicator version to try to reproduce the bug.

Revision history for this message
Pablo SEMINARIO (pabluk) wrote :

Thanks Ted, use a system dbus service seems an interesting idea. Maybe I'll try to test it.

Changed in screenkey:
status: New → Confirmed
importance: Undecided → Medium
Pablo SEMINARIO (pabluk)
Changed in screenkey:
assignee: nobody → Pablo Seminario (pabluk)
status: Confirmed → In Progress
Revision history for this message
Pablo SEMINARIO (pabluk) wrote :

I have ported the code needed to run as root to python-xlib and work fine without gksu.

Pablo SEMINARIO (pabluk)
Changed in screenkey:
status: In Progress → Fix Committed
Pablo SEMINARIO (pabluk)
Changed in screenkey:
status: Fix Committed → Fix Released
milestone: none → 0.2.0
Andrew (and471)
Changed in indicator-application:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.