mirror of
https://github.com/aluxnimm/outlookcaldavsynchronizer.git
synced 2026-04-25 11:05:56 +03:00
[GH-ISSUE #28] Attendees without email address lead to exception #485
Labels
No labels
1.0
1.0
1.0
2.0
Feature
Feature request
Google
Google Calendar
async
attachement
auto-migrated
auto-migrated
auto-migrated
bug
critical
enhancement
help wanted
implemented
pull-request
solved
solved
sourceforge
sourceforge
sourceforge
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/outlookcaldavsynchronizer#485
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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
@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
@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
@aluxnimm commented on GitHub (Mar 28, 2015):
Original comment by: nertsch