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

fix the returned permissions for webdav uploads #2179

Merged
merged 1 commit into from
Oct 19, 2021
Merged

fix the returned permissions for webdav uploads #2179

merged 1 commit into from
Oct 19, 2021

Conversation

wkloucek
Copy link
Contributor

@wkloucek wkloucek commented Oct 15, 2021 •
edited
Loading

Uploading a file gives now following permissions:

  • share (editor role): SDNVW
  • user home: RDNVW

fixes owncloud/ocis#2626

@wkloucek wkloucek requested a review from dragotin October 15, 2021 06:03
@wkloucek wkloucek mentioned this pull request Oct 15, 2021
Closed
dragotin
dragotin previously approved these changes Oct 18, 2021
View reviewed changes
Copy link
Contributor

@dragotin dragotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@wkloucek wkloucek marked this pull request as ready for review October 18, 2021 13:27
@wkloucek wkloucek requested a review from labkode as a code owner October 18, 2021 13:27
@wkloucek wkloucek requested a review from ishank011 October 18, 2021 13:29
@ishank011 ishank011 merged commit 2fcbe16 into cs3org:master Oct 19, 2021
gmgigi96 pushed a commit to cernbox/reva that referenced this pull request Oct 25, 2021
@wkloucek
fix the returned permissions for webdav uploads (cs3org#2179)
554fa05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ishank011 ishank011 ishank011 approved these changes

@micbar micbar micbar approved these changes

@dragotin dragotin dragotin left review comments

@labkode labkode Awaiting requested review from labkode

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[TUS] Generate correct Permissions Header
4 participants
@wkloucek @dragotin @micbar @ishank011