Skip to content
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

Bad url construction when multiples calendar-home-set. #139

Closed
flechilla opened this issue Jun 9, 2016 · 6 comments
Closed

Bad url construction when multiples calendar-home-set. #139

flechilla opened this issue Jun 9, 2016 · 6 comments

Comments

@flechilla
Copy link

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
Copy link
Owner

What server are you using? Never seen that but should be fixed if this is valid of course.

@flechilla
Copy link
Author

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
Copy link

evert commented 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
Copy link
Owner

ok thx for clarification. so for Autodiscovery we should check all available home-sets.

@aluxnimm
Copy link
Owner

fix will be in next release.

@aluxnimm
Copy link
Owner

included in release 2.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants