WebDAV service implementation for Kolab Files
Details
- Reviewers
mollekopf - Group Reviewers
Restricted Project - Commits
- rK15bf297f63ad: WebDAV Server
./phpunit
Diff Detail
- Repository
- rK kolab
- Branch
- dev/webdav-server
- Lint
Lint Skipped - Unit
No Test Coverage - Build Status
Buildable 55935 Build 19827: arc lint + arc unit
Event Timeline
Looks like a great start!
| src/app/Http/Middleware/ContentSecurityPolicy.php | ||
|---|---|---|
| 24 | I think these headers only make sense for browsers, so excluding webdav makes sense I think. | |
| src/config/services.php | ||
| 66 | The reasons I could see to go for either of them:
AFAIK sharing is handled differently in DAV (shared folders are represented within your own collections), so /dav/files seems ok? I don't think we have to keep backwards compatibility though. | |
| src/config/services.php | ||
|---|---|---|
| 66 | I just noticed that Kolab3 uses just /files. We currently have /files/.* in Cockpit, so we have a conflict to solve if we wanted to keep compatibility with Kolab3. Anyway, I think we should go for consistency with Cyrus DAV where calendars and addressbooks use /dav/calendars/user/<email> and /dav/addressbooks/user/<email>. So, I'd go for /dav/files/user/<email>. | |