Skip to content
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

directly query LDAP in ocis-reva #1249

Closed
synnove opened this issue Jul 2, 2020 · 3 comments
Closed

directly query LDAP in ocis-reva #1249

synnove opened this issue Jul 2, 2020 · 3 comments

Comments

@synnove
Copy link

synnove commented Jul 2, 2020

Currently ocis-reva appears to need nscd/nslcd running alongside it to query LDAP for user lookups. It would be interesting to try doing queries directly using a library instead.

@butonic
Copy link
Member

butonic commented Jul 2, 2020

yes, currently the eos driver uses the go stdlib to look up the numeric userid. AFAICT it is only needed to make the eos call with a numeric user and group id ...

since this is an implementation detail of the storage driver I'd love to leave the aspect out of cs3, however the storage needs to be able to be integrated with all users (including guest acounts). ocis uses the ocis-glauth to provide an ldap api with numeric ids for all users (including guests). It can be replaced with a dedicated ldap server, but it then needs to contain guest accounts as well. otherwise sharing with guest accounts is not possible.

I think the numeric id should not be part of the cs3 api, but we need to better document that there might be dependencies on ldap for storage drivers.

In any case making use of ldap (or the users service?) in the eos storage driver would allow running a single process in the eos storeage driver container ... that being said: caching user and group lookup using nslcd has its merits too ...

@butonic
Copy link
Member

butonic commented Jul 10, 2020

tracked in reva as cs3org/reva#964

@butonic butonic transferred this issue from owncloud/ocis-reva Jan 18, 2021
@refs
Copy link
Member

refs commented Jan 18, 2021

closing as superceding issue is closed (cs3org/reva#964)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants