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

Search sharee by username is broken #2999

Closed
fschade opened this issue Jan 20, 2022 · 4 comments
Closed

Search sharee by username is broken #2999

fschade opened this issue Jan 20, 2022 · 4 comments
Assignees

Comments

@fschade
Copy link
Contributor

fschade commented Jan 20, 2022

Describe the bug

Searching a sharee by displayname seems to be broken, the backend only considers the username when searching for users.

Steps to reproduce

  1. run ocis v1.16.0

  2. open webui

  3. share folder with string 'marie c' -> works

  4. run ocis from master

  5. open webui

  6. share folder with string 'marie c' -> does not work

request done by the frontend: https://localhost:9200/ocs/v2.php/apps/files_sharing/api/v1/sharees?search=marie%20c&itemType=folder&page=1&perPage=200&format=json

Expected behavior

find the user by displayname

Actual behavior

only considers the username when searching for a sharee

https://github.com/cs3org/reva/blob/3d065e97841af9abbeb416ec03ff39c550b8e2bd/pkg/user/manager/ldap/ldap.go#L286-L292

@fschade
Copy link
Contributor Author

fschade commented Jan 20, 2022

cc ^ @rhafer

@phil-davis
Copy link
Contributor

same as #2966 ?

@rhafer rhafer self-assigned this Jan 20, 2022
@rhafer
Copy link
Contributor

rhafer commented Jan 20, 2022

Yeah seems to be the same issue. Though it seems that my conclusion in #2966 is wrong. Need to take a closer look.

@fschade
Copy link
Contributor Author

fschade commented Jan 20, 2022

yup same, closing in favor of #2966

@fschade fschade closed this as completed Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants