Checkbox

Checkbox needs to use pkexec instead of gksu

Bug #1097816 reported by Brendan Donegan on 2013-01-09

24

This bug affects 3 people

Affects		Status	Importance	Assigned to	Milestone
	Checkbox	Fix Released	High	Daniel Manrique
	checkbox (Ubuntu)	Fix Released	High	Daniel Manrique

Bug Description

In conversation with several people, the consensus seems to be that gksu is deprecated for the desktop in favour of pkexec. We need to switch Checkbox over to it to avoid falling victim to bugs which then won't be fixed. There is already one present on the Nexus7 tablet which prevents the sudo password from being input

Tags:

Related branches

lp:~roadmr/checkbox/1097816-pkexec

Merged into lp:checkbox at revision 2219

Zygmunt Krynicki (community): Approve on 2013-07-01

Revision history for this message

Brendan Donegan (brendan-donegan) wrote on 2013-01-09:

#1

An initial stab at this didn't work, but I just need to look at it more closely to find a solution.

Changed in checkbox:
importance:	Undecided → Medium
status:	New → In Progress
assignee:	nobody → Brendan Donegan (brendan-donegan)

Revision history for this message

Zygmunt Krynicki (zyga) wrote on 2013-05-17:

#2

Can we produce a list of places (jobs, scripts, classes/methods) that depend on gksu

Revision history for this message

Daniel Manrique (roadmr) wrote on 2013-05-17:

#3

Yes, it's pretty easy, only the plugins/backend_info.py plugin uses sudo. The code is simple, but what it does is fork, and the forked process tries to superuser-execute the "backend" script. "superuser-execute" means trying to determine which sudo-invoker to use (it can choose between kdesudo, gksudo and plain sudo to do so).

Revision history for this message

Zygmunt Krynicki (zyga) wrote on 2013-05-17:

#4

OK so apparently we could use pkexec for that. With the same dual policy file we could ask one or no passwords when needing to restart as root

Daniel Manrique (roadmr) on 2013-06-06

tags:

added: checkbox-core

Revision history for this message

Brendan Donegan (brendan-donegan) wrote on 2013-06-15:

#5

We seem to know what to do for this, but I'm not going to prioritise working on this, since we are rapidly moving towards plainbox which already supports pkexec

Changed in checkbox:
status:	In Progress → Triaged
assignee:	Brendan Donegan (brendan-donegan) → nobody

Revision history for this message

Brendan Donegan (brendan-donegan) wrote on 2013-06-25:

#6

We might want to re prioritise this if Checkbox is going to still be in Saucy in it's current form (seems likely), since the Ubuntu release managers seem to want to be able to drop gksu - https://bugs.launchpad.net/ubuntu/+source/checkbox/+bug/1194195

Revision history for this message

Daniel Manrique (roadmr) wrote on 2013-06-26:

#7

Reprioritized to high :)

Changed in checkbox (Ubuntu):
importance:	Undecided → High
Changed in checkbox:
importance:	Medium → High

Daniel Manrique (roadmr) on 2013-06-27

Changed in checkbox:
status:	Triaged → In Progress
assignee:	nobody → Daniel Manrique (roadmr)
Changed in checkbox (Ubuntu):
status:	New → In Progress
assignee:	nobody → Daniel Manrique (roadmr)

Daniel Manrique (roadmr) on 2013-08-29

Changed in checkbox:
status:	In Progress → Fix Released
Changed in checkbox (Ubuntu):
status:	In Progress → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

Duplicates of this bug

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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