Skip to content

Commit 80169af

Browse files
committed
fix: prevent null token
Signed-off-by: skjnldsv <[email protected]>
1 parent 4fd2926 commit 80169af

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

lib/Listener/ShareLinkAccessedListener.php

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,15 @@
3333
use OCP\EventDispatcher\IEventListener;
3434
use OCP\IConfig;
3535
use OCP\Share\IManager;
36+
use Psr\Log\LoggerInterface;
3637

3738
/** @template-implements IEventListener<ShareLinkAccessedEvent> */
3839
class ShareLinkAccessedListener implements IEventListener {
3940
public function __construct(
4041
private IConfig $config,
4142
private IManager $manager,
4243
private LimitMapper $mapper,
44+
private LoggerInterface $logger
4345
) {
4446
}
4547

@@ -53,8 +55,12 @@ public function handle(Event $event): void {
5355
return;
5456
}
5557

56-
// Make sure we have a valid limit
5758
$token = $event->getShare()->getToken();
59+
if ($token === null) {
60+
return;
61+
}
62+
63+
// Make sure we have a valid limit
5864
try {
5965
$shareLimit = $this->mapper->get($token);
6066
$limit = $shareLimit->getLimit();
@@ -76,6 +82,8 @@ public function handle(Event $event): void {
7682
$this->mapper->update($shareLimit);
7783
} catch (DoesNotExistException $e) {
7884
// No limit is set, ignore
85+
} catch (\Exception $e) {
86+
$this->logger->error('Error while handling share link accessed event: ' . $e->getMessage());
7987
}
8088
}
8189
}

0 commit comments

Comments
 (0)