Hash event UID to make sure it's not too long for PushProvider

notifications

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

Author: tcitworld

apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php

@@ -93,14 +93,17 @@ public function send(VEvent $vevent,
 
 		$eventDetails = $this->extractEventDetails($vevent);
 		$eventDetails['calendar_displayname'] = $calendarDisplayName;
+		$eventUUID = (string) $vevent->UID;
+		// Empty Notification ObjectId will be catched by OC\Notification\Notification
+		$eventUUIDHash = $eventUUID ? hash('sha256', $eventUUID, false) : '';
 
 		foreach($users as $user) {
 			/** @var INotification $notification */
 			$notification = $this->manager->createNotification();
 			$notification->setApp(Application::APP_ID)
 				->setUser($user->getUID())
 				->setDateTime($this->timeFactory->getDateTime())
-				->setObject(Application::APP_ID, (string) $vevent->UID)
+				->setObject(Application::APP_ID, $eventUUIDHash)
 				->setSubject('calendar_reminder', [
 					'title' => $eventDetails['title'],
 					'start_atom' => $eventDetails['start_atom']

apps/dav/tests/unit/CalDAV/Reminder/NotificationProvider/PushProviderTest.php

@@ -189,7 +189,7 @@ private function createNotificationMock(string $uid, \DateTime $dt):INotificatio
 
 		$notification->expects($this->once())
 			->method('setObject')
-			->with('dav', 'uid1234')
+			->with('dav', hash('sha256', 'uid1234', false))
 			->willReturn($notification);
 
 		$notification->expects($this->once())