summary refs log tree commit diff
path: root/api/src
diff options
context:
space:
mode:
authorMadeline <46743919+MaddyUnderStars@users.noreply.github.com>2022-07-09 16:01:32 +1000
committerErkin Alp Güney <erkinalp9035@gmail.com>2022-07-09 09:45:28 +0300
commit2236cf593ccc2426b5bad5f88d0cd64e7e0e4c17 (patch)
tree60fd049fff2fb5281a371b3fcfc6c1a276980db5 /api/src
parentMerge pull request #783 from MaddyUnderStars/feat/multiUploads (diff)
downloadserver-2236cf593ccc2426b5bad5f88d0cd64e7e0e4c17.tar.xz
* Make max website preview download size for embeds configurable
* Fix Message.update call throwing 'Cannot query across one-to-many for property attachments'
Diffstat (limited to 'api/src')
-rw-r--r--api/src/util/handlers/Message.ts9
1 files changed, 6 insertions, 3 deletions
diff --git a/api/src/util/handlers/Message.ts b/api/src/util/handlers/Message.ts
index e9f0ac55..48f87dfe 100644
--- a/api/src/util/handlers/Message.ts
+++ b/api/src/util/handlers/Message.ts
@@ -38,7 +38,7 @@ const DEFAULT_FETCH_OPTIONS: any = {
 	headers: {
 		"user-agent": "Mozilla/5.0 (compatible; Fosscord/1.0; +https://github.com/fosscord/fosscord)"
 	},
-	size: 1024 * 1024 * 1,
+	// size: 1024 * 1024 * 5, 	// grabbed from config later
 	compress: true,
 	method: "GET"
 };
@@ -154,7 +154,10 @@ export async function postHandleMessage(message: Message) {
 
 	for (const link of links) {
 		try {
-			const request = await fetch(link, DEFAULT_FETCH_OPTIONS);
+			const request = await fetch(link, {
+				...DEFAULT_FETCH_OPTIONS,
+				size: Config.get().limits.message.maxEmbedDownloadSize,
+			});
 
 			const text = await request.text();
 			const $ = cheerio.load(text);
@@ -191,7 +194,7 @@ export async function postHandleMessage(message: Message) {
 			channel_id: message.channel_id,
 			data
 		} as MessageUpdateEvent),
-		Message.update({ id: message.id, channel_id: message.channel_id }, data)
+		Message.update({ id: message.id, channel_id: message.channel_id }, { embeds: data.embeds })
 	]);
 }