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.

Sign up for GitHub

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

Jump to bottom

Missing Securely Viewable permission in the REPORT response #9607

Closed
ScharfViktor opened this issue Jul 15, 2024 · 3 comments · Fixed by #9638
Closed

Missing Securely Viewable permission in the REPORT response #9607

ScharfViktor opened this issue Jul 15, 2024 · 3 comments · Fixed by #9638
Assignees
@2403905
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@ScharfViktor
Copy link
Contributor

related #9608

we have X: Securely Viewable in the webdav https://owncloud.dev/apis/http/webdav/#supported-webdav-properties
but REPORT response doesn't contain this permission

Steps:

  • admin shares file/folder with files with secure view role
  • recipient searches shared file

Expected:
oc:permissionsSM</oc:permissions> contains X

Actual:
REPORT doesn't contain X

Screenshot 2024-07-15 at 16 21 10

PROPFIND works correct
Screenshot 2024-07-15 at 16 22 41
@ScharfViktor ScharfViktor changed the title Missing Securely Viewable permission in the webdav Missing Securely Viewable permission in the REPORT response Jul 15, 2024
@micbar
Copy link
Contributor

micbar commented Jul 15, 2024

@butonic @aduffeck @kobergj

It seems to be a big problem, that the ocdav and search WebDAV implementations are split up 🙈

@micbar micbar added the Priority:p2-high Escalation, on top of current planning, release blocker label Jul 15, 2024
@ScharfViktor
Copy link
Contributor Author

ScharfViktor commented Jul 15, 2024 •
edited
Loading

@2403905 2403905 self-assigned this Jul 17, 2024
@2403905 2403905 mentioned this issue Jul 17, 2024
Merged
9 tasks
@AlexAndBear
Copy link
Contributor

Will probably fix: #9608

This was referenced Jul 19, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@2403905 2403905

Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Infinite Scale Team Board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a missing SecureView permission attribute in the REPORT response 2403905/ocis
4 participants
@micbar @2403905 @AlexAndBear @ScharfViktor