nextcloud / passman

🔐 Open source password manager with Nextcloud integration
https://passman.cc
GNU Affero General Public License v3.0
786 stars 108 forks source link

LDAP users are displayed by their internal UID instead of their user names #254

Open budachst opened 7 years ago

budachst commented 7 years ago

When sharing a credential with LDAP users, the sharing info does not display the actual user/account name, but nextcloud's internal UID, which makes this very cumbersone in a LDAP environment.

It actually looks like this:

admin -> local user 1125a389-6f4c-4c5a-8b1d-8f3ac86e738c -> LDAP user

This will make it quite difficult to keep up with one's shared credentials, when running in larger setup.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/42174967-ldap-users-are-displayed-by-their-internal-uid-instead-of-their-user-names?utm_campaign=plugin&utm_content=tracker%2F44880056&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F44880056&utm_medium=issues&utm_source=github).
brantje commented 7 years ago

Thanks for reporting. Unfortunately we don't have an ldap setup, and don't have the time to setup one either. Because it's purely visual i'll assign a low prio to this.

budachst commented 7 years ago

Sure - if I can be of any help, let me know.

brantje commented 7 years ago

Yes, do the LDAP users exist in the user table from nextcloud? Also, the places where they shown might come in handy.

budachst commented 7 years ago

There are three mapping tables in the database:

oc_ldap_group_mapping oc_ldap_group_members oc_ldap_user_mapping

I'd think, that you will mostly be interested in the oc_ldap_user_mapping at this time. The structure looks like this:

ldap_dn owncloud_name directory_uuid

These will map LDAP users to their resp. local user id. Places, where these are shown is e.g. the standard sharing dialog, when sharing a link or other item.

brantje commented 7 years ago

Nextcloud has classes for getting users, we need to use those i think.

budachst commented 7 years ago

It surely has - unfortuanetly I am not a dev, so I can't be of more help in this field, but I can perform some testing, if that'd help.

brantje commented 7 years ago

I've setup a local ldap installation, will start debugging this week i hope.

As far i know at the following places the usernames appear

If i missed some places, feel free to add them

mrPsycho commented 7 years ago

here is field with ID. screen shot 2017-06-19 at 20 54 09

agamez commented 5 years ago

I'm using nextcloud 16.0.1 and LDAP sharing simply doesn't work. I can't find any of the users using their name, but neither by using the internal ID. Is there any information I can provide to help solve this?

Or maybe some indication on what to look for in the code, as I'm not familiar at all with nextcloud's and its apps source code.

InsOpDe commented 1 year ago

We experience the same issue