fix: update external storage mounts on login

Signed-off-by: Robin Appelman <[email protected]>

Author: icewind1991

apps/files_external/lib/AppInfo/Application.php

@@ -58,6 +58,7 @@
 use OCP\Group\Events\GroupDeletedEvent;
 use OCP\Group\Events\UserAddedEvent;
 use OCP\Group\Events\UserRemovedEvent;
+use OCP\User\Events\PostLoginEvent;
 use OCP\User\Events\UserCreatedEvent;
 use OCP\User\Events\UserDeletedEvent;
 
@@ -87,6 +88,7 @@ public function register(IRegistrationContext $context): void {
 		$context->registerEventListener(UserCreatedEvent::class, MountCacheService::class);
 		$context->registerEventListener(UserAddedEvent::class, MountCacheService::class);
 		$context->registerEventListener(UserRemovedEvent::class, MountCacheService::class);
+		$context->registerEventListener(PostLoginEvent::class, MountCacheService::class);
 
 		$context->registerConfigLexicon(ConfigLexicon::class);
 	}

apps/files_external/lib/Service/MountCacheService.php

@@ -28,12 +28,13 @@
 use OCP\IGroupManager;
 use OCP\IUser;
 use OCP\IUserManager;
+use OCP\User\Events\PostLoginEvent;
 use OCP\User\Events\UserCreatedEvent;
 
 /**
  * Listens to config events and update the mounts for the applicable users
  *
- * @template-implements IEventListener<StorageCreatedEvent|StorageDeletedEvent|StorageUpdatedEvent|BeforeGroupDeletedEvent|UserCreatedEvent|UserAddedEvent|UserRemovedEvent|Event>
+ * @template-implements IEventListener<StorageCreatedEvent|StorageDeletedEvent|StorageUpdatedEvent|BeforeGroupDeletedEvent|UserCreatedEvent|UserAddedEvent|UserRemovedEvent|PostLoginEvent|Event>
  */
 class MountCacheService implements IEventListener {
 	private CappedMemoryCache $storageRootCache;
@@ -70,6 +71,9 @@ public function handle(Event $event): void {
 		if ($event instanceof UserCreatedEvent) {
 			$this->addUser($event->getUser());
 		}
+		if ($event instanceof PostLoginEvent) {
+			$this->onLogin($event->getUser());
+		}
 	}
 
 
@@ -221,4 +225,14 @@ private function addUser(IUser $user): void {
 			$this->registerForUser($user, $storage);
 		}
 	}
+
+	/**
+	 * Since storage config can rely on login credentials, we might need to update the config
+	 */
+	private function onLogin(IUser $user): void {
+		$storages = $this->storagesService->getAllGlobalStorages();
+		foreach ($storages as $storage) {
+			$this->registerForUser($user, $storage);
+		}
+	}
 }