Skip to content

Ignore empty CalDAV entry to allow syncing to continue - #526

Open
foolswood wants to merge 1 commit into
lfos:pufrom
foolswood:pu
Open

Ignore empty CalDAV entry to allow syncing to continue#526
foolswood wants to merge 1 commit into
lfos:pufrom
foolswood:pu

Conversation

@foolswood

Copy link
Copy Markdown

I don't entirely know why the CalDAV server I was using came back with a response like this, but failing to encode the None as UTF-8 didn't feel like desirable behaviour.

If we just ignore it and continue then the sync works great, so that seemed like the simplest thing to do.

Don't know if this is affecting anyone else, but presumably it could and hopefully the change isn't too scary.

@foolswood foolswood changed the title Ignore empty calendar entry to allow syncing to continue Ignore empty CalDAV entry to allow syncing to continue Apr 11, 2026
@foolswood

Copy link
Copy Markdown
Author

Realised it would probably be better to show a minimal example response that triggers this so you can see what's actually happening.

<?xml version="1.0" encoding="UTF-8"?>
<D:multistatus xmlns:D="DAV:" xmlns:CR="urn:ietf:params:xml:ns:carddav" xmlns:A="http://apple.com/ns/ical/" xmlns:K="Kolab:" xmlns:CS="http://calendarserver.org/ns/" xmlns:C="urn:ietf:params:xml:ns:caldav">
  <D:response>
    <D:href>/calendars/nope@notgivingthat.away/calendar/</D:href>
    <D:propstat>
      <D:prop>
        <D:getetag>1728205163</D:getetag>
      </D:prop>
      <D:status>HTTP/1.1 200 OK</D:status>
    </D:propstat>
    <D:propstat>
      <D:prop>
        <C:calendar-data xmlns:C="urn:ietf:params:xml:ns:caldav"/>
      </D:prop>
      <D:status>HTTP/1.1 404 Not Found</D:status>
    </D:propstat>
  </D:response>
</D:multistatus>

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