From fbde8f978b5fbac3b0c083f0fe6331b4ac5d84a8 Mon Sep 17 00:00:00 2001
From: Madeline <46743919+MaddyUnderStars@users.noreply.github.com>
Date: Mon, 5 Dec 2022 19:29:50 +1100
Subject: Fix private messages being returned when no channel_id provided in
 search

---
 src/api/routes/guilds/#guild_id/messages/search.ts | 20 +++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

(limited to 'src/api/routes/guilds')

diff --git a/src/api/routes/guilds/#guild_id/messages/search.ts b/src/api/routes/guilds/#guild_id/messages/search.ts
index f2d8087e..ccee59f7 100644
--- a/src/api/routes/guilds/#guild_id/messages/search.ts
+++ b/src/api/routes/guilds/#guild_id/messages/search.ts
@@ -1,8 +1,8 @@
 import { Request, Response, Router } from "express";
 import { route } from "@fosscord/api";
-import { getPermission, FieldErrors, Message } from "@fosscord/util";
+import { getPermission, FieldErrors, Message, Channel } from "@fosscord/util";
 import { HTTPError } from "lambert-server";
-import { FindManyOptions, Like } from "typeorm";
+import { FindManyOptions, In, Like } from "typeorm";
 
 const router: Router = Router();
 
@@ -38,7 +38,7 @@ router.get("/", route({}), async (req: Request, res: Response) => {
 	const permissions = await getPermission(
 		req.user_id,
 		req.params.guild_id,
-		channel_id as string,
+		channel_id as string | undefined,
 	);
 	permissions.hasThrow("VIEW_CHANNEL");
 	if (!permissions.has("READ_MESSAGE_HISTORY"))
@@ -70,6 +70,20 @@ router.get("/", route({}), async (req: Request, res: Response) => {
 	};
 	//@ts-ignore
 	if (channel_id) query.where!.channel = { id: channel_id };
+	else {
+		// get all channel IDs that this user can access
+		const channels = await Channel.find({ where: { guild_id: req.params.guild_id }, select: ["id"] });
+		const ids = [];
+
+		for (var channel of channels) {
+			const perm = await getPermission(req.user_id, req.params.guild_id, channel.id);
+			if (!perm.has("VIEW_CHANNEL") || !perm.has("READ_MESSAGE_HISTORY")) continue;
+			ids.push(channel.id);
+		}
+
+		//@ts-ignore
+		query.where!.channel = { id: In(ids) };
+	}
 	//@ts-ignore
 	if (author_id) query.where!.author = { id: author_id };
 	//@ts-ignore
-- 
cgit 1.4.1