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

Workarounds for broken caldav servers #247

Merged
merged 3 commits into from
Jan 15, 2023
Merged

Workarounds for broken caldav servers #247

merged 3 commits into from
Jan 15, 2023

Conversation

tobixen
Copy link
Member

@tobixen tobixen commented Jan 14, 2023

  • Zimbra may apparently create events with both DTSTART, DTEND and DURATION, which is a breach of the standard. A simple workaround has been implemented, just throwing away redundant information. No check done on weather the DURATION matches DTEND-DTSTART (it did in the ical data I got from Zimbra), so this one may potentially throw away information. (I just wrote a message to the organizer asking if the original ical data was actually produced internally Zimbra or by some other client).
  • I've been fighting a bit with some proprietary system in Skipped event with missing 'vevent' property (Home Assistant) #201 that have some partial and slightly non-compliant support for caldav - and made some partial workaround for loading content from it.
  • One extra unrelated commit enhancing the tests code has been piggy-backed on this pull request.

@tobixen tobixen merged commit 22011e4 into master Jan 15, 2023
@tobixen
Copy link
Member Author

tobixen commented Jan 15, 2023

I shouldn't do such things in the late evenings ... commit 22011e4 and 5ad53ce should have been included in this pull request :-)

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

Successfully merging this pull request may close these issues.

1 participant