403 move forbidden: better description
Open
Nobody has claimed this yet.
1. to develop
enhancement
feature: dav
feature: sharing
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
- have a folder "withTest", which you share with user "test"
- have a folder "fromAdmin", which Admin shared with you
- now try to move "fromAdmin" into "withTest"
- on UI you get a "forbidden message", without an explanation
- on webdav you get:
<?xml version="1.0" encoding="utf-8"?>
<d:error
xmlns:d="DAV:"
xmlns:s="http://sabredav.org/ns">
<s:exception>Sabre\DAV\Exception\Forbidden</s:exception>
<s:message/>
</d:error>
This should have a (localized) message, so that clients can show this message instead of a generic 403 error code
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the described move from the Admin-shared folder into the user-shared folder through the UI and WebDAV, then trace the forbidden response path. Done means the forbidden condition returns a localized explanatory message instead of an empty WebDAV message or generic UI error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- api, backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100