Ticket #25 (closed defect: obsolete)

Opened 5 years ago

Last modified 4 years ago

Journal hardcodes USB mounting as UID=500

Reported by: morgs Owned by: tomeu
Priority: major Milestone:
Component: sugar Version: 0.82.x
Severity: Blocker Keywords:
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Sucrose 0.82.x

 https://bugs.launchpad.net/ubuntu/+source/sugar-journal-activity/+bug/297479

It appears that there is a hard coded UID for mounting USB flash drives.

In Journal.activity/volumesmanager.py
--
MOUNT_OPTION_UID = 500
--

This should use the UID of the actual user on the system who is running Sugar. Note: there may be more than one user active in a multiseat/thin client system.
On a one user Ubuntu system, the UID of the user is 1000. On a multi user system, the UIDs will vary.

Change History

Changed 5 years ago by mungewell

The same code has be moved to:
[sugar] / src / jarabe / model / volume.py

Although there is a note in [sugar] / src / jarabe / journal / volumestoolbar.py that the mounting of USB drives has been disabled for now.

Changed 5 years ago by mungewell

Brief reading around this subject... might ConsoleKit and PolicyKit be useful in this situation, if you want to add the dependencies.

Otherwise the volumestoolbar should use the current UID and hope that there is no-one else active on the machine.

Changed 4 years ago by marcopg

Yeah sounds like we should be using CK here.

Changed 4 years ago by tomeu

  • owner changed from marcopg to tomeu
  • status changed from new to assigned

Changed 4 years ago by tomeu

  • version set to 0.82.x

That's only 0.82, in 0.83 we are already using GIO for that and I tested as working on non-XOs.

Changed 4 years ago by gregdek

  • status changed from assigned to closed
  • distribution set to Unspecified
  • resolution set to fixed
  • severity set to Blocker
  • status_field set to Unconfimed

Since this is fixed in the current release, and since we do not currently have manpower to backport fixes to older releases, we are closing this bug.

Changed 4 years ago by gregdek

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 4 years ago by gregdek

  • status changed from reopened to closed
  • resolution set to obsolete

Closing properly as "obsolete".

Note: See TracTickets for help on using tickets.