-
Notifications
You must be signed in to change notification settings - Fork 186
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
[Sharing NG] permissions for a drive are missing its members #8352
Comments
scratch that. I think we should not be returning permissions with the drives response at all on the v1beta1 implementation. This should be added to the driveItem related endpoints. (MS Graph does not specific the
|
In order to work with (e.g. get/delete) permissions granted to space we need to give them a stable id. As the CS3 API don't provide an id we generate it base on the id of the identity that the permission applies to. For users we use "u:<userid>" for groups "g:<groupid>". Closes: owncloud#8352
In order to work with (e.g. get/delete) permissions granted to space we need to give them a stable id. As the CS3 API don't provide an id we generate it base on the id of the identity that the permission applies to. For users we use "u:<userid>" for groups "g:<groupid>". Closes: owncloud#8352
In order to work with (e.g. get/delete) permissions granted to space we need to give them a stable id. As the CS3 API don't provide an id we generate it base on the id of the identity that the permission applies to. For users we use "u:<userid>" for groups "g:<groupid>". Closes: #8352
Describe the bug
When fetching the permissions of a drive they are missing its members.
Steps to reproduce
curl "https://localhost:9200/graph/v1beta1/drives/{drive-id}/items/{item-id}/permissions" -u admin:admin -vk | jq
Expected behavior
All members and links should be represented as a permission in the server's response.
Actual behavior
Members are missing, the server only returns the links.
cc @rhafer
The text was updated successfully, but these errors were encountered: