mirror of
https://github.com/aluxnimm/outlookcaldavsynchronizer.git
synced 2026-04-26 03:25:48 +03:00
[GH-ISSUE #139] Bad url construction when multiples calendar-home-set. #136
Labels
No labels
1.0
1.0
1.0
2.0
Feature
Feature request
Google
Google Calendar
async
attachement
auto-migrated
auto-migrated
auto-migrated
bug
critical
enhancement
help wanted
implemented
pull-request
solved
solved
sourceforge
sourceforge
sourceforge
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/outlookcaldavsynchronizer#136
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @flechilla on GitHub (Jun 9, 2016).
Original GitHub issue: https://github.com/aluxnimm/outlookcaldavsynchronizer/issues/139
If the principal has multiple DAV:calendar-home-set, so the server returns something like this:
<C:calendar-home-set xmlns:C="urn:ietf:params:xml:ns:caldav" xmlns:D="DAV:"> <D:href>/collections/users/aff31/</D:href> <D:href>/collections/groups/public/</D:href> </C:calendar-home-set>So the client concatenate the href of calendar-home-set, so it makes a request to the url
PROPFIND /collections/users/aff31//collections/groups/public/ HTTP/1.1.It should not be like that, it should perfoms the PROFIND method to the hrefs in different requests.
@aluxnimm commented on GitHub (Jun 10, 2016):
What server are you using? Never seen that but should be fixed if this is valid of course.
@flechilla commented on GitHub (Jun 13, 2016):
I'm using my own server implementation. That doesn't matters because my server is returning the response that I mention before, which is OK. Maybe you haven't seen the bug before because the principal that you have tried doesn't have more than 1 calendar-home.
Sorry for not reply before.
@evert commented on GitHub (Jun 13, 2016):
The issue itself is valid from a CalDAV protocol perspective, but also almost nobody actually supports multiple calendar homes (I don't know anyone who does).
@aluxnimm commented on GitHub (Jun 13, 2016):
ok thx for clarification. so for Autodiscovery we should check all available home-sets.
@aluxnimm commented on GitHub (Jun 15, 2016):
fix will be in next release.
@aluxnimm commented on GitHub (Jun 25, 2016):
included in release 2.2.0.