Merge pull request #24964 from nextcloud/bugfix/noid/internal-server-error-when-importing-broken-ics-file

Don't throw a 500 when importing a broken ics reminder file
This commit is contained in:
Roeland Jago Douma 2021-01-05 12:57:00 +01:00 committed by GitHub
commit 50b380e85b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions

View File

@ -40,6 +40,7 @@ use OCP\IUserManager;
use Sabre\VObject;
use Sabre\VObject\Component\VAlarm;
use Sabre\VObject\Component\VEvent;
use Sabre\VObject\InvalidDataException;
use Sabre\VObject\ParseException;
use Sabre\VObject\Recur\EventIterator;
use Sabre\VObject\Recur\NoInstancesException;
@ -274,7 +275,11 @@ class ReminderService {
continue;
}
$triggerTime = $valarm->getEffectiveTriggerTime();
try {
$triggerTime = $valarm->getEffectiveTriggerTime();
} catch (InvalidDataException $e) {
continue;
}
// If effective trigger time is in the past
// just skip and generate for next event