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

All-day EXDATEs are not applied to recurring date/time events #44600

Open
rfc2822 opened this issue Apr 1, 2024 · 3 comments
Open

All-day EXDATEs are not applied to recurring date/time events #44600

rfc2822 opened this issue Apr 1, 2024 · 3 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: caldav Related to CalDAV internals

Comments

@rfc2822
Copy link
Contributor

rfc2822 commented Apr 1, 2024

Some calendar clients generate recurring events with date and time, plus all-day EXDATEs:

BEGIN:VEVENT
UID:527f3d37-ed76-4c6a-8aae-bf6d71a4ad09
DTSTART;TZID=Europe/Vienna:20240317T100000          ← DATE-TIME value
DTEND;TZID=Europe/Vienna:20240317T150000
STATUS:CONFIRMED
SUMMARY:Test Event
RRULE:FREQ=WEEKLY;BYDAY=SU;UNTIL=20240526T080000Z
EXDATE;VALUE=DATE:20240331           ← DATE value
EXDATE;VALUE=DATE:20240421
EXDATE;VALUE=DATE:20240505
EXDATE;VALUE=DATE:20240519
EXDATE;VALUE=DATE:20240324
END:VEVENT
END:VCALENDAR

As I understand it, this is not a RFC 5545-conforming way to specify EXDATEs for the given event at 10:00. However it's clear that the recurring event should not occur on those dates.

Currently, Nextcloud expands to the given EXDATEs, too. It would be more compatible to omit the recurring events on the dates given by the EXDATEs.

DAVx5 handles this by rewriting all RDATE/EXDATE entries to the value type of DTSTART:

  • If DTSTART is a DATE and EXDATE a DATE-TIME, just drop the EXDATE time and make it all-day.
  • If DTSTART is a DATE-TIME and EXDATE is a DATE, take the time (+ time zone) of DTSTART and the date of EXDATE to generate the new EXDATE.
@rfc2822 rfc2822 added 0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement labels Apr 1, 2024
@kesselb
Copy link
Contributor

kesselb commented Apr 1, 2024

cc @ChristophWurst @miaulalala

@miaulalala
Copy link
Contributor

Do you know which clients produce these?

@rfc2822
Copy link
Contributor Author

rfc2822 commented Apr 2, 2024

In my case I have created the event with Gnome Evolution (but as I know there are others)

@joshtrichards joshtrichards added the feature: caldav Related to CalDAV internals label Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: caldav Related to CalDAV internals
Projects
None yet
Development

No branches or pull requests

4 participants