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

Implement GetUserByClaim for LDAP user driver #1088

Merged
merged 1 commit into from
Aug 17, 2020

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Aug 13, 2020

The LDAP user driver can now fetch users by a single claim / attribute. Use an attributefilter like (&(objectclass=posixAccount)({{attr}}={{value}})) in the driver section.

partly fixes owncloud/ocis#444

@butonic
Copy link
Contributor Author

butonic commented Aug 13, 2020

I need to add the uid and gid to the opaque user properties ...

@butonic butonic marked this pull request as draft August 13, 2020 11:10
@butonic butonic force-pushed the ldap-get-by-claim branch from b699c39 to 43f40cd Compare August 13, 2020 11:18
@butonic butonic marked this pull request as ready for review August 13, 2020 11:18
@butonic
Copy link
Contributor Author

butonic commented Aug 13, 2020

With this PR and owncloud/ocis-proxy#89 I can share files with other users when running ocis on top of eos.

@butonic butonic self-assigned this Aug 13, 2020
@butonic butonic added the feature New feature label Aug 13, 2020
@labkode labkode merged commit 5ec5513 into cs3org:master Aug 17, 2020
glpatcern pushed a commit to glpatcern/reva that referenced this pull request Aug 24, 2020
@butonic butonic deleted the ldap-get-by-claim branch October 1, 2021 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

With EOS storage backend, the new WebDAV API doesn't work on #409
2 participants