Guest Session should not show if guest session was disabled in the display manager

Bug #835084 reported by Jeremy Bícha
54
This bug affects 10 people
Affects Status Importance Assigned to Milestone
Light Display Manager
Fix Released
High
Robert Ancell
Session Menu
Fix Released
Medium
Conor Curran
Unity Foundations
Fix Released
Medium
Conor Curran
indicator-session (Ubuntu)
Fix Released
High
Unassigned
lightdm (Ubuntu)
Fix Released
High
Unassigned

Bug Description

A user may disable the guest session because they only want authenticated users using the computer; indicator-session should respect this choice.

1. Add "allow-guest=false" to /etc/lightdm/lightdm.conf
   (Or alternatively, if using gdm, don't install gdm-guest-session)
2. Log out and log back in.
3. The user menu should no longer have the Guest Session button because all it does is lock the screen.

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: indicator-session 0.3.3.2-0ubuntu1
ProcVersionSignature: Ubuntu 3.0.0-9.14-generic 3.0.3
Uname: Linux 3.0.0-9-generic x86_64
Architecture: amd64
Date: Fri Aug 26 17:03:28 2011
SourcePackage: indicator-session
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Jeremy Bícha (jbicha) wrote :
Changed in indicator-session (Ubuntu):
status: New → Confirmed
Revision history for this message
Andy Rogers (andy-rogers) wrote :

Also, if gdm-guest-session is not installed, the Guest Session menu option still also appears in the indiciator session menu.

Conor Curran (cjcurran)
Changed in indicator-session:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Conor Curran (cjcurran)
Revision history for this message
Robert Ancell (robert-ancell) wrote :

There should be a 'HasGuest' property on the D-Bus object for each seat, opening lightdm bug on this.

Changed in lightdm:
status: New → Triaged
importance: Undecided → High
Changed in lightdm (Ubuntu):
status: New → Triaged
importance: Undecided → High
Conor Curran (cjcurran)
Changed in unity-foundations:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Conor Curran (cjcurran)
Conor Curran (cjcurran)
Changed in indicator-session (Ubuntu):
status: Confirmed → In Progress
Changed in lightdm:
assignee: nobody → Robert Ancell (robert-ancell)
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Fixed in lightdm 0.9.5

Changed in lightdm:
status: Triaged → Fix Committed
Changed in lightdm (Ubuntu):
status: Triaged → Fix Committed
Changed in lightdm:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package lightdm - 0.9.5-0ubuntu1

---------------
lightdm (0.9.5-0ubuntu1) oneiric; urgency=low

  [ Steve Langasek ]
  * Update upstart job to use current start condition lifted from gdm,
    and to check runlevels instead of /proc/cmdline so it's possible to
    start lightdm after switching out of recovery mode. LP: #803513.

  [ Didier Roche ]
  * debian/patches/01_transition_gnome_ubuntu_desktop.patch:
    natty-oneiric session name upgrade:
    - transition from unity-2d -> ubuntu-2d
    - transition from gnome-2d -> gnome-fallback

  [ Robert Ancell ]
  * New upstream release:
   [0.9.5]
    - Use accounts service in the daemon if it is available
    - Correctly load seat type in multi seat configuration
    - Add display-setup, session-setup and session-cleanup scripting hooks
    - Fix cancel button in GTK greeter (LP: #819240)
    - Fix line through GTK greeter menu items
    - Exit daemon if a seat fails which has exit-on-failure set to true
    - Add HasGuestAccount property to seat D-Bus interface (LP: #835084)
    - Fix XDMCP authorization
    - Update man file
    - Emit upstart events (LP: #715094)
   [0.9.4]
    - lightdm-set-defaults enables tweaking the default session and chosen
      greeter for lightdm. This is useful for derivatives waiting to not ship
      the whole configuration file of lightdm
    - Fix crash in GTK+ greeter when a user is added
    - Move xsessions-directory and xgreeters-directory from [SeatDefaults] to
      [LightDM]. This is a configuration break, but making it on the assumption
      that these settings are not likely to have been overridden.
    - Fix fallback from org.freedesktop.Accounts to passwd format
    - Fix duplicate user entries being shown when using passwd file
    - Add AddSeat D-Bus method for adding dynamic seats
    - Added a dm-tool program that allows user switching and adding seats
    - Allow remote X servers, e.g. launched using dm-tool add-nested-seat
    - Fix bug where sessions were started when the greeter quit and the user
      hadn't been authorized.
    - Fix bug where sessions used the seat bus name
    - Don't allow autologin-username to be set to empty
    - Fix bug where PAM session was not opened before writing to home directory
    - Fix crash when failing to write X authority
 -- Robert Ancell <email address hidden> Wed, 07 Sep 2011 16:09:53 +1000

Changed in lightdm (Ubuntu):
status: Fix Committed → Fix Released
Conor Curran (cjcurran)
Changed in indicator-session:
status: Triaged → In Progress
Changed in unity-foundations:
status: Triaged → In Progress
Conor Curran (cjcurran)
Changed in indicator-session:
milestone: none → 0.3.5
Changed in unity-foundations:
milestone: none → oneiric-beta-2
Conor Curran (cjcurran)
Changed in indicator-session:
status: In Progress → Fix Committed
Changed in unity-foundations:
status: In Progress → Fix Committed
Conor Curran (cjcurran)
Changed in indicator-session:
status: Fix Committed → Fix Released
Changed in unity-foundations:
status: Fix Committed → Fix Released
Changed in indicator-session (Ubuntu):
status: In Progress → Fix Committed
importance: Undecided → High
Conor Curran (cjcurran)
Changed in indicator-session (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

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