fix(logging): simplify hub retrieval and role checks in getHubForUser…

… method
interchatapp · Feb 27, 2025 · 0bcfe00 · 0bcfe00
1 parent b9d8802
commit 0bcfe00
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/commands/Main/hub/config/logging.ts b/src/commands/Main/hub/config/logging.ts
@@ -325,12 +325,13 @@ export default class HubConfigLoggingSubcommand extends BaseCommand {
 
   private async getHubForUser(ctx: Context): Promise<HubManager | null> {
     const hubName = ctx.options.getString('hub', true);
-    const hubs = await this.hubService.findHubsByName(hubName, {
-      insensitive: true,
-      ownerId: ctx.user.id,
-    });
+    const [hub] = await this.hubService.findHubsByName(hubName, { insensitive: true });
+
+    if (!hub || !(await executeHubRoleChecksAndReply(hub, ctx, { checkIfManager: true }))) {
+      return null;
+    }
 
-    return hubs[0] ?? null;
+    return hub;
   }
 
   private buildComponents(