Merge pull request #32987 from nextcloud/backport/stable23/31771

[stable23] Fix hook encryption with cron job
nextcloud · Jul 7, 2022 · c7e9a59 · c7e9a59
2 parents 7f41e72 + 003b2a4
commit c7e9a59
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 14 deletions.
diff --git a/apps/files_sharing/lib/ExpireSharesJob.php b/apps/files_sharing/lib/ExpireSharesJob.php
@@ -85,7 +85,7 @@ public function run($argument) {
 				)
 			);
 
-		$shares = $qb->execute();
+		$shares = $qb->executeQuery();
 		while ($share = $shares->fetch()) {
 			if ((int)$share['share_type'] === IShare::TYPE_LINK) {
 				$id = 'ocinternal';

diff --git a/lib/private/Encryption/HookManager.php b/lib/private/Encryption/HookManager.php
@@ -28,36 +28,45 @@
 use Psr\Log\LoggerInterface;
 
 class HookManager {
-	/**
-	 * @var Update
-	 */
-	private static $updater;
+	/** @var ?Update */
+	private static $updater = null;
 
-	public static function postShared($params) {
+	public static function postShared($params): void {
 		self::getUpdate()->postShared($params);
 	}
-	public static function postUnshared($params) {
-		self::getUpdate()->postUnshared($params);
+	public static function postUnshared($params): void {
+		// In case the unsharing happens in a background job, we don't have
+		// a session and we load instead the user from the UserManager
+		$path = Filesystem::getPath($params['fileSource']);
+		$owner = Filesystem::getOwner($path);
+		self::getUpdate($owner)->postUnshared($params);
 	}
 
-	public static function postRename($params) {
+	public static function postRename($params): void {
 		self::getUpdate()->postRename($params);
 	}
 
-	public static function postRestore($params) {
+	public static function postRestore($params): void {
 		self::getUpdate()->postRestore($params);
 	}
 
-	/**
-	 * @return Update
-	 */
-	private static function getUpdate() {
+	private static function getUpdate(?string $owner = null): Update {
 		if (is_null(self::$updater)) {
 			$user = \OC::$server->getUserSession()->getUser();
+			if (!$user && $owner) {
+				$user = \OC::$server->getUserManager()->get($owner);
+			}
+			if (!$user) {
+				throw new \Exception("Inconsistent data, File unshared, but owner not found. Should not happen");
+			}
+
 			$uid = '';
 			if ($user) {
 				$uid = $user->getUID();
 			}
+
+			\OC_Util::setupFS($uid);
+
 			self::$updater = new Update(
 				new View(),
 				new Util(