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

Groups displayed as UUID on the Account page #7554

Closed
rhafer opened this issue Aug 30, 2022 · 5 comments · Fixed by #7568
Closed

Groups displayed as UUID on the Account page #7554

rhafer opened this issue Aug 30, 2022 · 5 comments · Fixed by #7568
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@rhafer
Copy link

rhafer commented Aug 30, 2022

This is bascially for the web part of owncloud/ocis#4283

Describe the bug

image

Expected behavior

Value from the attribute set by LDAP_GROUP_SCHEMA_DISPLAYNAME should be used instead. The default should be cn already but even it if the env var is explicitly set to cn the ID is displayed.

Setup

Container (owncloud/ocis:2.0.0-beta.5) + External OpenLDAP

@rhafer
Copy link
Author

rhafer commented Aug 30, 2022

@kulmann with owncloud/ocis#4476 the graph API will support /me/?$expand=memberOf, now web "just" needs to use that on the Account page. (AFAIK currently it's still using an 'ocs' query.

@kulmann
Copy link
Member

kulmann commented Aug 31, 2022

Needs a capability so that I know when to request the groups via graph or via ocs. ;-) Are there any plans on some kind of a graph capability?

@kulmann
Copy link
Member

kulmann commented Aug 31, 2022

Ticket for graph capability in ocis: owncloud/ocis#4481

@AlexAndBear
Copy link
Contributor

Note, we currently use capabilities.spaces.enabled
in packages/web-runtime/src/services/auth/userManager.ts:165

@kulmann
Copy link
Member

kulmann commented Sep 1, 2022

I'd be fine with using that until we have the graph capability. 👍

@AlexAndBear AlexAndBear added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Sep 1, 2022
@kulmann kulmann moved this to Done in Web Support Board Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants