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

Cannot share with user and group having same name #35488

Closed
phil-davis opened this issue Jun 11, 2019 · 6 comments · Fixed by #36359
Closed

Cannot share with user and group having same name #35488

phil-davis opened this issue Jun 11, 2019 · 6 comments · Fixed by #36359
Assignees
Labels
Milestone

Comments

@phil-davis
Copy link
Contributor

Steps to reproduce

  1. create user abcdef and group abcdef
  2. as another user, create a folder
  3. using the webUI, share the folder with group abcdef
  4. now also share the folder with user abcdef

Expected behaviour

Both shares should be created.

Actual behaviour

When sharing at step 4, webUI says "Path already shared with this user"

Similarly, if I share with the user first, then try to share with the group, webUI says "Path already shared with this group"

Note: this happens with v10.1.0, v10.2.0, current stable10 and with PR #35397 - so it is nothing new.

Server configuration

Operating system:
Ubuntu 18.04

Web server:
PHP dev server

Database:
MySQL

PHP version:
7.2

ownCloud version: (see ownCloud admin page)
Current stable10

@phil-davis
Copy link
Contributor Author

Note: this happens if the group name and the user name are identical.

e.g. create group AbcDef and user abcdef or ABCDEF and everything is OK
but create group AbcDef and user AbcDef and there is a problem.

I expect that this is webUI-specific. But I will make some test scenarios for the sharing API to see if it is also broken.

@phil-davis
Copy link
Contributor Author

See also issue #36069 - @HanaGemela reports that this bad behaviour is also noticed from the desktop client. So I guess that there is some real problem in the server API.

@phil-davis
Copy link
Contributor Author

I added the QA-team label to remind us to make some API tests.
It would be good to really verify if there is an API problem, or if the problem is only in the UI.

@dpakach
Copy link
Contributor

dpakach commented Oct 17, 2019

Acceptance tests demonstrating the bug in #36283

@dpakach dpakach removed their assignment Oct 21, 2019
@phil-davis
Copy link
Contributor Author

Removed QA-team label - tests have been added to demonstrate the bug in the sharing API.
Over to devs to prioritize and schedule a fix...
@micbar

@micbar
Copy link
Contributor

micbar commented Nov 11, 2019

Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants