Treat local and remote users differently. We try to use /etc/passwd as a heuristic for whether a user is 'local' or 'remote'. Also don't run userdel for remote users. Should we be deleting the user's home directory manually for remote users if remove_files is set?
Created attachment 62964 [details] [review] Add User.Local dbus property This property describes whether the user account is local to the machine or not, and thus can be managed with usermod, userdel and so on. We currently implement this as a heuristic. If it's found in /etc/passwd then we treat it as 'local'
Created attachment 62965 [details] [review] Don't use userdel for deletion of remote users
Comment on attachment 62965 [details] [review] Don't use userdel for deletion of remote users Moving this patch to another bug.
Created attachment 65555 [details] [review] Rename user_local_xxxx() to user_xxxx() Rename user_local_xxxx() to user_xxxx() * Since User* applies to remote users too, and the 'local' terminology can be confusing.
Created attachment 65556 [details] [review] Add User.Local dbus property Updated the patch so it works correctly in user cases
Comment on attachment 65555 [details] [review] Rename user_local_xxxx() to user_xxxx() Review of attachment 65555 [details] [review]: ----------------------------------------------------------------- seems fine, though now daemon is inconsistent. Pushed.
Comment on attachment 65556 [details] [review] Add User.Local dbus property Review of attachment 65556 [details] [review]: ----------------------------------------------------------------- pushed.
I think all is done here.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.