[GH-ISSUE #28] Attendees without email address lead to exception #17

Closed
opened 2026-02-25 20:30:43 +03:00 by kerem · 3 comments
Owner

Originally created by @aluxnimm on GitHub (Mar 27, 2015).
Original GitHub issue: https://github.com/aluxnimm/outlookcaldavsynchronizer/issues/28

Originally assigned to: @aluxnimm on GitHub.

Some Attendees have no value, e.g.
ATTENDEE;PARTSTAT=NEEDS-ACTION;CN=Max Mustermann:invalid:nomail

So this leads to NullreferenceException
if (!indexByEmailAddresses.TryGetValue (attendee.Value.ToString(), out targetRecipient))
{

Only the Cn is available and should be used in that case.

Original comment by: aluxnimm

Original Ticket: outlookcaldavsynchronizer/17

Originally created by @aluxnimm on GitHub (Mar 27, 2015). Original GitHub issue: https://github.com/aluxnimm/outlookcaldavsynchronizer/issues/28 Originally assigned to: @aluxnimm on GitHub. Some Attendees have no value, e.g. ATTENDEE;PARTSTAT=NEEDS-ACTION;CN=Max Mustermann:invalid:nomail So this leads to NullreferenceException if (!indexByEmailAddresses.TryGetValue (attendee.Value.ToString(), out targetRecipient)) { Only the Cn is available and should be used in that case. Original comment by: aluxnimm Original Ticket: [outlookcaldavsynchronizer/17](https://sourceforge.net/p/outlookcaldavsynchronizer/17)
Author
Owner

@aluxnimm commented on GitHub (Mar 27, 2015):

The code can be easily modified to process Atendees without email, but this is not RFC conform.
Sogo throw away invalid attendees, but Outlook accepts attendees without email.

Should invalid Atendees be generated and accepted, or should the synchonizer ignore invalid attendees ?

Original comment by: nertsch

<!-- gh-comment-id:126312560 --> @aluxnimm commented on GitHub (Mar 27, 2015): The code can be easily modified to process Atendees without email, but this is not RFC conform. Sogo throw away invalid attendees, but Outlook accepts attendees without email. Should invalid Atendees be generated and accepted, or should the synchonizer ignore invalid attendees ? Original comment by: nertsch
Author
Owner

@aluxnimm commented on GitHub (Mar 28, 2015):

the icalendar validator

http://severinghaus.org/projects/icv/

accepts it.

so it should be synced. maybe it could be tried to find the cn in the address book and get the email?

Original comment by: aluxnimm

<!-- gh-comment-id:126312561 --> @aluxnimm commented on GitHub (Mar 28, 2015): the icalendar validator http://severinghaus.org/projects/icv/ accepts it. so it should be synced. maybe it could be tried to find the cn in the address book and get the email? Original comment by: aluxnimm
Author
Owner

@aluxnimm commented on GitHub (Mar 28, 2015):

  • labels: --> solved
  • status: open --> closed

Original comment by: nertsch

<!-- gh-comment-id:126312563 --> @aluxnimm commented on GitHub (Mar 28, 2015): - **labels**: --> solved - **status**: open --> closed Original comment by: nertsch
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/outlookcaldavsynchronizer#17
No description provided.