-
Notifications
You must be signed in to change notification settings - Fork 229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing API/interface for changing fullname (gecos) and shell #985
Comments
Correct me if I'm wrong, but the idea would be to provide the |
Yes, you are correct. Just few notes: We need to be root to do this so binary must have setuid bit. Therefore the GUI (wayland/X11) app will still use some lightweight wrapper like Also we will have just one code for manipulating |
This is kind of the standard procedure for these cases. I'm fine with it. @hallyn please check this ticket and share your thoughts. |
@ikerexxe I'm good with this. Did you want to open a pr for the change? |
@hallyn thanks for the response. We might be able to work on this. I wanted to know if this proposal is acceptable before putting an effort into it. |
The shadow-util have binaries for setting user details and shell (
chfn
andchsh
).Those function are not exported. As a result the GUI for user settings can't actually use them, because chfn and chsh are designed for human interaction. Most of those tools uses
usermode
package witch providesuserhelper
binary anduserhelper
useslibuser
for this purpose.Libuser
re-implements chfn and chsh and manipulates/etc/passwd
on its own. Unfortunatelylibuser
project is not actively developed any more (I'm the maintainer) and so the tools mentioned above depend on obsolete code.It would be great to make functions like
update_gecos
andupdate_shell
public. This will allow us to get rid of libuser dependency.Just for completeness -
libuser
supposed to hide differences between LDAP and shadow users, this is kind of working but not for all LDAP schemes and there is no effort to catch up the technological debt.Tomáš
The text was updated successfully, but these errors were encountered: