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

Add UserTypeProperty to get user type from ldap user entry #3744

Merged

Conversation

Excds
Copy link

@Excds Excds commented Mar 23, 2023

Adding an LDAP attribute UserTypeProperty to be able to distinguish between member and guest users.

@update-docs
Copy link

update-docs bot commented Mar 23, 2023

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@Excds Excds force-pushed the add_UserTypeProperty_to_get_user_type_from_ldap branch 2 times, most recently from f1a10e0 to d55045a Compare March 23, 2023 11:37
@Excds Excds requested review from glpatcern and a team as code owners March 23, 2023 11:37
rhafer
rhafer previously requested changes Mar 23, 2023
Copy link
Contributor

@rhafer rhafer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the full URL of the PR in the changelog

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
@butonic butonic force-pushed the add_UserTypeProperty_to_get_user_type_from_ldap branch from d55045a to 86b2705 Compare March 24, 2023 10:08
@butonic butonic requested a review from rhafer March 24, 2023 10:08
pkg/utils/ldap/identity.go Outdated Show resolved Hide resolved
Co-authored-by: kobergj <juliankoberg@googlemail.com>
@micbar micbar merged commit 36c2ef9 into cs3org:edge Mar 24, 2023
@Excds Excds deleted the add_UserTypeProperty_to_get_user_type_from_ldap branch March 24, 2023 10:45
@micbar micbar mentioned this pull request May 3, 2023
89 tasks
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

Successfully merging this pull request may close these issues.

5 participants