Merge pull request #1957 from nextcloud/enh/perf-read

Emit read hook instead of fopen for admin_audit

lib/TokenManager.php

@@ -22,6 +22,7 @@
 namespace OCA\Richdocuments;
 
 use InvalidArgumentException;
+use OC\Files\Filesystem;
 use OCA\Richdocuments\Db\Direct;
 use OCA\Richdocuments\Db\WopiMapper;
 use OCA\Richdocuments\Db\Wopi;
@@ -191,8 +192,11 @@ public function getToken($fileId, $shareToken = null, $editoruid = null, $direct
 		}
 
 		// force read operation to trigger possible audit logging
-		$fp = $file->fopen('r');
-		fclose($fp);
+		\OC_Hook::emit(
+			Filesystem::CLASSNAME,
+			Filesystem::signal_read,
+			[Filesystem::signal_param_path => $file->getPath()]
+		);
 
 		$serverHost = $this->urlGenerator->getAbsoluteURL('/');
 		$guestName = $this->userId === null ? $this->prepareGuestName($this->helper->getGuestNameFromCookie()) : null;