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

folder share with edit permission - error with WOPI #8834

Closed
dj4oC opened this issue Apr 11, 2024 · 10 comments
Closed

folder share with edit permission - error with WOPI #8834

dj4oC opened this issue Apr 11, 2024 · 10 comments
Labels

Comments

@dj4oC
Copy link
Contributor

dj4oC commented Apr 11, 2024

oCIS 5.0.0

Steps to reproduce:

  1. create a folder
  2. create a txt and a docx/pptx/xlsx document in that folder
  3. share that folder with edit permissions
  4. the share receiver can edit the txt document
  5. the share receiver can not edit the docx/pptx/xlsx document
  6. the share receiver can not create a new docx/pptx/xlsx document
  7. the share receiver can create a new txt document
  8. the share receiver can not upload a docx/pptx/xlsx document
  9. the share receiver can not upload a txt document
Screen.Recording.2024-04-11.at.08.43.29.mov
@dj4oC dj4oC added Type:Bug Priority:p1-urgent Consider a hotfix release with only that fix labels Apr 11, 2024
@wkloucek
Copy link
Contributor

I'm not sure if we run the WOPI validator on public links in the meantime (#5111, #8712)

@micbar
Copy link
Contributor

micbar commented Apr 11, 2024

Public link?

@dj4oC
Copy link
Contributor Author

dj4oC commented Apr 11, 2024

Public link?
yes

@micbar
Copy link
Contributor

micbar commented Apr 11, 2024

Duplicate of #8691 and https://github.com/owncloud/enterprise/issues/6558

We are already on it.

Thanks for reporting.

EE issue from customer has Sev3, P3. Aligning.

@micbar micbar added Priority:p3-medium Normal priority and removed Priority:p1-urgent Consider a hotfix release with only that fix labels Apr 11, 2024
@micbar
Copy link
Contributor

micbar commented Apr 11, 2024

I'm not sure if we run the WOPI validator on public links in the meantime (#5111, #8712)

Not related to WOPI. Was related to app/open and app/new.

WOPI testing plans are here #8712 for the new collaboration service.

@dj4oC
Copy link
Contributor Author

dj4oC commented Apr 12, 2024

New finding: Also upload of office documents throughs an error after upload.
reload page shows the file correctly.
upload of text files work fine.

@micbar
Copy link
Contributor

micbar commented Apr 15, 2024

New finding: Also upload of office documents throughs an error after upload.

Not reproducable on the vanilla product.

@micbar micbar moved this from Qualification to In progress in Infinite Scale Team Board Apr 16, 2024
@rhafer
Copy link
Contributor

rhafer commented Apr 16, 2024

With #8870 and cs3org/reva#4632 I merged I can no longer reproduce any of the above mentioned issues.

Backport to stable-5.0 is in the works.

@rhafer rhafer closed this as completed Apr 16, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Infinite Scale Team Board Apr 16, 2024
@micbar micbar moved this from Done to Needs Tests in Infinite Scale Team Board Apr 17, 2024
@micbar
Copy link
Contributor

micbar commented Apr 17, 2024

@ScharfViktor moved to "needs tests"

@ScharfViktor
Copy link
Contributor

@ScharfViktor moved to "needs tests"

thanks. we'll create test for that
already tested it manualy. works fine

same issue: #8691

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

No branches or pull requests

5 participants