Don't tolerated malformed vcards (was: Malformed(?) VCARD causes hang)

[flash666@…]

In the SetupGuide? part of the test process involved the manual creation of a VCARD without a VERSION tag. I have changed it now, because doing so and attempting a kdepim-file sync results in msynctool hanging. I had a couple of test contacts in KDE, and an empty directory. I successfully synched the two. Then I added a vcard to the directory that looked like this:

BEGIN:VCARD
N:test user
END:VCARD

msynctool then hung on the next run while attempting to send this to the kde member. After changing the VCARD to look like this:

BEGIN:VCARD
VERSION:2.1
N:test user
END:VCARD

the synch was fine. Log files all at ​http://www.bus-times.org.uk/dump

According to the VCARD 2.1 spec (​http://www.imc.org/pdi/vcard-21.txt):

Version
This property specifies the identifier corresponding to the highest version number of the vCard Specification supported by the implementation that created the vCard object. The value of this property must be 2.1 to correspond to this specification.. 
This property is identified by the property name VERSION. The following is an example of this property:
VERSION:2.1
Support for this property is mandatory for implementations conforming to this specification. This property must appear within the vCard data stream.

So I suppose cards of this type should be considered malformed, logged and ignored.

[pmarat]

this should be fixed with r1696

[pmarat]

r1709 reopens this ticket

[dgollub]

I will try to workaround this for 0.21 .. maybe osync_filter can help.

[dgollub]

In OpenSync? 0.33 it doesn't cause anymore a hang. The vformat tolerates malformed entries and adds required VEERSION information... the vcard* -> xmlforamt-contact should do a sanity check if the vcard is malformed (missing version header) and throw an error ... instead of tolerating.

dani@K51:~/Sync/file> cat 1/contact/malformat.vcf 
BEGIN:VCARD
N:test user
END:VCARD
dani@K51:~/Sync/file> cat 2/contact/malformat.vcf 
BEGIN:VCARD
VERSION:2.1
N:test user
dani@K51:~/Sync/file> 

This entry got synced with OpenSync? 0.33 from member 1 to member 2. Note: tail is missing of synced entry (directory 2).