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

[QA] Share owner cannot delete other user's files #1126

Closed
jnweiger opened this issue Dec 17, 2020 · 3 comments
Closed

[QA] Share owner cannot delete other user's files #1126

jnweiger opened this issue Dec 17, 2020 · 3 comments
Labels
Category:Defect Existing functionality is not working as expected Type:Bug

Comments

@jnweiger
Copy link
Contributor

Deployed ocis rc8 via ocis/deployments/examples/ocis_traefik/
https://ocis.rc.owncloud.works

(From @refs )

  • login as admin
  • create folder A
  • share folder A with Marie as Editor
  • login as Marie
  • upload a file to folder A
  • login as admin
  • navigate to folder A and try to delete the file that Marie uploaded to /a
    • will fail. BAD
  • navigate one dir up (../)
  • delete the shared folder with Marie
  • will succeed. Okayish

Expected behaviour: deleting a file from a share user should work for the owner.
Other share users can delete hers, just not the owner.

Maybe releated to: #1118

@jnweiger jnweiger changed the title [QA] share owner cannot delete other user's files (direclty) [QA] share owner cannot delete other user's files Dec 17, 2020
@individual-it
Copy link
Member

individual-it commented Dec 21, 2020

  • ToDo QA-team: check if there are API tests for this case, if not add

@swoichha swoichha self-assigned this Dec 21, 2020
@swoichha swoichha removed their assignment Dec 28, 2020
@settings settings bot removed the QA-Team label Jan 12, 2021
@refs refs changed the title [QA] share owner cannot delete other user's files [QA] Share owner cannot delete other user's files Jan 13, 2021
@refs refs added Category:Defect Existing functionality is not working as expected QA-Team Type:Bug labels Jan 13, 2021
@settings settings bot removed the QA-Team label Jan 29, 2021
@settings settings bot removed the p3-medium label Apr 7, 2021
@ScharfViktor
Copy link
Contributor

re-tested with local server oCIS version 1.11.0

It works correctly. there is no mention in the /expected-failures
We can close

@phil-davis
Copy link
Contributor

Agree - this is a "simple" sharing sequence that works nowadays.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Defect Existing functionality is not working as expected Type:Bug
Projects
None yet
Development

No branches or pull requests

6 participants