-
Notifications
You must be signed in to change notification settings - Fork 187
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
Two shares with the same name cause sync issue #7118
Comments
Server issue I'd say… What version? Listing Space content
Listing folder content
|
reproducable in two ways:
curl --location --request PROPFIND 'https://cloud.ocis.test/remote.php/dav/spaces/a0ca6a90-a365-4782-871e-d44447bbc668!a0ca6a90-a365-4782-871e-d44447bbc668/Neuer Ordner (1)/' \
--header 'Depth: 1' \
--header 'Authorization: Basic bWFyaWU6cmFkaW9hY3Rpdml0eQ=='
<?xml version="1.0" encoding="UTF-8"?>
<d:error xmlns:d="DAV" xmlns:s="http://sabredav.org/ns">
<s:exception>Sabre\DAV\Exception\NotFound</s:exception>
<s:message>Resource not found</s:message>
</d:error>
Hm I get all possible responses: a 404, a 207 and a 500 A 500 has this suspicious line (the missing ̀
|
a successful response
log
|
a 404 not found
log
|
a 500 log
|
Yeah so the storageprovider logs this
The request already seems to have a wrong path in the reference:
|
sometimes the func buildReferenceInShare(ref *provider.Reference, s *collaboration.ReceivedShare) *provider.Reference {
path := ref.Path
if isShareJailRoot(ref.ResourceId) {
// we need to cut off the mountpoint from the path in the request reference
path = utils.MakeRelativePath(strings.TrimPrefix(strings.TrimPrefix(path, "./"), s.MountPoint.Path))
}
return &provider.Reference{
ResourceId: s.Share.ResourceId,
Path: path,
}
} it should be |
for _, receivedShare := range lsRes.Shares {
if receivedShare.State != collaboration.ShareState_SHARE_STATE_ACCEPTED {
continue
}
if strings.HasPrefix(strings.TrimPrefix(ref.Path, "./"), receivedShare.MountPoint.Path) {
return receivedShare, lsRes.Status, nil
}
} this also returns the share if the mountpoint is a prefix of the path, which |
oh well ... I guess we should do another bugfix release right after 4.0.0 |
Desktop client version
ownCloud 5.0.0.11822-daily20230821 65017f
windows-10.0.19045
For 2 shares with same name
Before force sync
one share(test-share) is synced locally
but in another share(test-share(1)) error occurs:
Error transferring https://192.168.56.1:9200/dav/spaces/a0ca6a90-a365-4782-871e-d44447bbc668$a0ca6a90-a365-4782-871e-d44447bbc668/test-share (1) - server replied: Not Found
❔ownCloud.log
Originally posted by @Salipa-Gurung in owncloud/client#11048 (comment)
The text was updated successfully, but these errors were encountered: