Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

change default ocdav namespace #68

Merged
merged 1 commit into from
Jan 30, 2020

Conversation

butonic
Copy link
Member

@butonic butonic commented Jan 30, 2020

Currently, cross storage etag propagation is not yet implemented, which prevents the desktop client from detecting changes via the PROPFIND to /. / is managed by the root storage provider which is independend of the home and oc storage providers. If a file changes in /home/foo, the etag change will only be propagated to the root of the home storage provider.

This change jails users into the /home namespace, and allows configuring the namespace to use for the two webdav endpoints using the new environment variable WEBDAV_NAMESPACE_JAIL which affects both endpoints /dav/files and /webdav.

This will allow us to focus on getting a single storage driver like eos or owncloud tested and better resembles what owncloud 10 does.

To get back the global namespace, which ultimately is the goal, just set the above environment variable to /.

@butonic butonic self-assigned this Jan 30, 2020
@butonic butonic added the enhancement New feature or request label Jan 30, 2020
@butonic butonic requested a review from refs January 30, 2020 09:06
Copy link
Member

@DeepDiver1975 DeepDiver1975 left a comment

Choose a reason for hiding this comment

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

this needs to go into ocis-webdav at the end right?

pkg/flagset/frontend.go Outdated Show resolved Hide resolved
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
@butonic butonic force-pushed the change-default-namespace branch from d65891a to fbf131c Compare January 30, 2020 10:17
@DeepDiver1975 DeepDiver1975 merged commit 23613e4 into owncloud:master Jan 30, 2020
@butonic butonic deleted the change-default-namespace branch January 30, 2020 11:03
@butonic butonic mentioned this pull request Feb 4, 2020
1 task
Copy link
Member

@refs refs left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants