Skip to content
This repository has been archived by the owner. It is now read-only.

REQUEST-STATUS value incorrectly encoded #189

Closed
macosforgebot opened this issue Jan 23, 2008 · 5 comments
Closed

REQUEST-STATUS value incorrectly encoded #189

macosforgebot opened this issue Jan 23, 2008 · 5 comments

Comments

@macosforgebot
Copy link

@macosforgebot macosforgebot commented Jan 23, 2008

@cyrusdaboo originally submitted this as ticket:228


vobject is incorrectly encoding the REQUEST-STATUS value when set by the calendar server:

REQUEST-STATUS:2.0\; Success.

There should not be a \ before the ;.

The problem here is that this property has a very specific format that does not match the usual iCalendar text or text list behaviors that vobject provided. However vobject treats the property as a test list. So even though the value is set without the \, vobject encodes it with a \.

The fix for this is to special case this property as a text list - but one that uses ; as a separator rather than ,.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Jan 23, 2008

@wsanchez originally submitted this as comment:1:⁠ticket:228


This has not apparently caused any problems for iCal when handling the server's auto-responses.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Jan 23, 2008

@wsanchez originally submitted this as comment:2:⁠ticket:228

  • Milestone changed from 1.2 to 1.x
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 14, 2008

@wsanchez originally submitted this as comment:3:⁠ticket:228


vobject is updated to r208 in trunk. Dunno if this is fixed in that version.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 29, 2008

@m0rgen originally submitted this as comment:4:⁠ticket:228

  • Status changed from new to closed
  • Resolution set to fixed

Committed a patch to vobject in svn.osafoundation.org (rev 209)

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Oct 28, 2008

@wsanchez originally submitted this as comment:5:⁠ticket:228

  • Milestone changed from CalendarServer-1.x to CalendarServer-2.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.