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

Trying to change user from LDAP finished with an error #98

Open
artiomn opened this Issue May 5, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@artiomn

artiomn commented May 5, 2018

OMV\ExecException: Failed to execute command 'export PATH=/bin:/sbin:/usr/bin:/usr/sbin:/usr/local/bin:/usr/local/sbin; export LANG=C; usermod --gid 'users' --shell '/bin/bash' --comment 'Sergey' --groups 'ldap_users' 'sergey' 2>&1' with exit code '6': usermod: user 'sergey' does not exist in /etc/passwd in /usr/share/php/openmediavault/system/process.inc:175
Stack trace:
#0 /usr/share/openmediavault/engined/rpc/usermgmt.inc(559): OMV\System\Process->execute()
#1 [internal function]: OMVRpcServiceUserMgmt->setUser(Array, Array)
#2 /usr/share/php/openmediavault/rpc/serviceabstract.inc(123): call_user_func_array(Array, Array)
#3 /usr/share/php/openmediavault/rpc/rpc.inc(86): OMV\Rpc\ServiceAbstract->callMethod('setUser', Array, Array)
#4 /usr/sbin/omv-engined(536): OMV\Rpc\Rpc::call('UserMgmt', 'setUser', Array, Array, 1)
#5 {main}

sergey doesn't exist in the passwd, because he was received from the LDAP server.

@donhwy

This comment has been minimized.

donhwy commented May 8, 2018

I don't think omv is the proper place to manage your ldap users. It would make it nearly impossible to maintain for all possible ldap servers. Maybe add some type of disclaimer to only maintain local users?

@artiomn

This comment has been minimized.

artiomn commented May 8, 2018

I don't think omv is the proper place to manage your ldap users.

Now you have an interface, that doesn't work properly.

It would make it nearly impossible to maintain for all possible ldap servers.

You can support only OpenLDAP, for example, and then improve the plugin, if necessary.

Maybe add some type of disclaimer to only maintain local users?

Maybe, but improve interface and provide a ability to manage users is better.

@votdev

This comment has been minimized.

Collaborator

votdev commented May 8, 2018

Contributions are welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment