Merge pull request #338 from nextcloud/backport/328/stable29

[stable29] fix: prevent null token

lib/Listener/ShareLinkAccessedListener.php

@@ -33,13 +33,15 @@
 use OCP\EventDispatcher\IEventListener;
 use OCP\IConfig;
 use OCP\Share\IManager;
+use Psr\Log\LoggerInterface;
 
 /** @template-implements IEventListener<ShareLinkAccessedEvent> */
 class ShareLinkAccessedListener implements IEventListener {
 	public function __construct(
 		private IConfig $config,
 		private IManager $manager,
 		private LimitMapper $mapper,
+		private LoggerInterface $logger
 	) {
 	}
 
@@ -53,8 +55,12 @@ public function handle(Event $event): void {
 			return;
 		}
 
-		// Make sure we have a valid limit
 		$token = $event->getShare()->getToken();
+		if ($token === null) {
+			return;
+		}
+
+		// Make sure we have a valid limit
 		try {
 			$shareLimit = $this->mapper->get($token);
 			$limit = $shareLimit->getLimit();
@@ -76,6 +82,8 @@ public function handle(Event $event): void {
 			$this->mapper->update($shareLimit);
 		} catch (DoesNotExistException $e) {
 			// No limit is set, ignore
+		} catch (\Exception $e) {
+			$this->logger->error('Error while handling share link accessed event: ' . $e->getMessage());
 		}
 	}
 }