Launchpad has imported 18 comments from the remote bug at https://bugs.freedesktop.org/show_bug.cgi?id=56729.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2012-11-03T21:51:30+00:00 Matthew Monaco wrote: The hardcoded default_excludes is really annoying. It's about time we had support for a configuration file. The attached patch set includes this support. It adds support for [UserList]/Excludes which is shipped with the list of users currently hardcoded in default_excludes. It also adds support for [UserList]/MinUID with -- as far as I can tell -- a common default of 1000. However, this is now easy for distributions to modify. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/36 ------------------------------------------------------------------------ On 2012-11-03T21:52:08+00:00 Matthew Monaco wrote: Created attachment 69500 0001-util.h-add-missing-headers.patch Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/37 ------------------------------------------------------------------------ On 2012-11-03T21:52:32+00:00 Matthew Monaco wrote: Created attachment 69501 0002-cfg-initial-support-for-a-configuration-file.patch Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/38 ------------------------------------------------------------------------ On 2012-11-03T21:52:51+00:00 Matthew Monaco wrote: Created attachment 69502 0003-cfg-replace-default_excludes-with-configuration-file.patch Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/39 ------------------------------------------------------------------------ On 2012-11-03T21:54:03+00:00 Matthew Monaco wrote: Created attachment 69503 0004-cfg-add-option-for-minimum-uid.patch Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/40 ------------------------------------------------------------------------ On 2012-11-05T17:00:33+00:00 Rstrode wrote: Hey, I'd prefer if you added metadata to each users config file instead of introduciting a new file. Also, one thing to think about is accountsservice is used in various places to get a list of users 1) the login screen 2) the control center panel maybe it would make sense to show the user in the panel even when not in the login screen? So we may want two keys. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/41 ------------------------------------------------------------------------ On 2012-11-05T18:05:46+00:00 Matthew Monaco wrote: Ray, I agree that more keys can be added, but all this does is pull a really arbitrary hard-coded list out of the binary into a config file. (And re-add Minimum UID because it made sense but no-one could agree on a value). As for the per-user config file, I gotta disagree with you. Those are in /var/lib/... which doesn't really suggest they are intended to be done by hand. Also, the minimum UID is global behavior. I could see adding an option like Visibility to the per-user config which can be always on, no-cache, or off, .... But why block which IMO is more of a bug-fix because it also makes you want a new feature. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/42 ------------------------------------------------------------------------ On 2012-11-12T19:53:36+00:00 Rstrode wrote: I think we also want to keep a separation between admin configuration and distro configuration. We should probably have two files, one in data dir and one in /etc/accountsservice/ Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/43 ------------------------------------------------------------------------ On 2012-11-12T19:53:59+00:00 Rstrode wrote: (by data dir, i mean $(datadir), i.e. /usr/share/accountsservice) Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/44 ------------------------------------------------------------------------ On 2012-11-13T20:10:56+00:00 Matthew Monaco wrote: I agree, but I didn't want to put too much time into it before I knew it might be accepted. Do you think that a single file in /usr that's overridable in /etc is enough? I could see a case for a config.d, lexographically sorted set of config files so that a package could drop in an exclude for a specific username that it installs. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/45 ------------------------------------------------------------------------ On 2012-11-13T20:42:25+00:00 Rstrode wrote: I think distros should be able to have their list which includes adm, lp, and nobody, and admins should be able to augment the list with their own users that could include tarball installs of 3rd party software and specific real users they want to conceal. I'd say the distro file would go in /usr and the admin file in /etc. Note, it would be bad if the admin file overrode the list instead of augmenting it. I don't think config.d makes sense. we have /var/lib/AccountsService/users/foo already that stores metadata about a specific user foo. that metadata could have "exclude from user list" or "exclude from control center panel" in it, I suppose. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/46 ------------------------------------------------------------------------ On 2012-11-13T22:20:16+00:00 Matthew Monaco wrote: Ok Ray. Hopefully I can finish it off this weekend. (The only thing about /var is, it seems like it's bad form to install files there, but lets not get hung up on that). Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/47 ------------------------------------------------------------------------ On 2012-11-14T16:25:23+00:00 Rstrode wrote: well, the files are usually written there by accountsservice in response to dbus calls (e.g. from changes to gnome-control-center or whatever). I guess we could introduce a accountsctl cli utility or so for managing the files, so the admin doesn't have to edit the /var files directly and doesn't have to use dbus-send. Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/48 ------------------------------------------------------------------------ On 2012-11-23T01:15:29+00:00 Matthew Monaco wrote: Ok Ray, I still need to add the config file under /usr/lib, but check it out so far [1]. This takes "Exclude" from either the config file(s) or the per-user file under /var/lib. I tried to make a distinction between a system account and an excluded account. The check for a system account is similar to what the daemon_local_is_excluded() function did. I don't think the check for HAVE_GETUSERSHELL was working because I have those functions but HAVE_GETUSERSHELL wasn't defined. I just removed the ifdef for now... I still am including the MinimumUID setting. I think it would be best to default this to SYS_UID_MAX from /etc/login.defs, but I don't know the best way to retreive that value. [1] https://github.com/mmonaco/accountsservice/tree/exclude-v2 Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/49 ------------------------------------------------------------------------ On 2012-11-28T03:39:28+00:00 Matthew Monaco wrote: https://github.com/mmonaco/accountsservice/commits/exclude-v3 Add's a config file to /lib/... and /etc/... Excluded= and MinimumUID config options. MinimumUID in /etc overrides /lib. Exclude in /etc is combind with /lib Add's an excluded property to the User type and the DBus handlers for manipulating this type Admin users are not cached. Excluded users are still cached, but they are not returned by ListCachedUsers... I think the best thing would be to just add a new method such as ListGreeterUsers() and get the DMs to use this instead. Thoughts? Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/50 ------------------------------------------------------------------------ On 2012-11-29T22:38:40+00:00 Rstrode wrote: hey, i just got back from a vacation away from computer (since the 16th), i'll look over your stuff soon! Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/51 ------------------------------------------------------------------------ On 2012-12-06T19:32:39+00:00 Rstrode wrote: *** Bug 50103 has been marked as a duplicate of this bug. *** Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/52 ------------------------------------------------------------------------ On 2012-12-06T19:33:39+00:00 Rstrode wrote: *** Bug 41908 has been marked as a duplicate of this bug. *** Reply at: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/857651/comments/54 ** Changed in: accountsservice Status: Unknown => Confirmed ** Changed in: accountsservice Importance: Unknown => Medium -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to accountsservice in Ubuntu. https://bugs.launchpad.net/bugs/857651 Title: Unable to hide users from login screen / user switcher Status in D-Bus interfaces for querying and manipulating user account information: Confirmed Status in “accountsservice” package in Ubuntu: Triaged Status in “lightdm” package in Ubuntu: Triaged Bug description: Users that I have appended to the 'hidden-users' field in /etc/lightdm/users.conf are not actually hidden. They are still listed on the login screen and in Unity's user switching menu. ProblemType: Bug DistroRelease: Ubuntu 11.10 Package: lightdm 0.9.7-0ubuntu1 ProcVersionSignature: Ubuntu 3.0.0-11.18-generic 3.0.4 Uname: Linux 3.0.0-11-generic x86_64 ApportVersion: 1.23-0ubuntu1 Architecture: amd64 Date: Fri Sep 23 11:44:29 2011 InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Beta amd64 (20110413) SourcePackage: lightdm UpgradeStatus: Upgraded to oneiric on 2011-09-23 (0 days ago) mtime.conffile..etc.lightdm.users.conf: 2011-09-23T08:46:55.039175 To manage notifications about this bug go to: https://bugs.launchpad.net/accountsservice/+bug/857651/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : [email protected] Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
