diff options
author | Flam3rboy <34555296+Flam3rboy@users.noreply.github.com> | 2021-09-01 11:27:52 +0200 |
---|---|---|
committer | Flam3rboy <34555296+Flam3rboy@users.noreply.github.com> | 2021-09-01 11:27:52 +0200 |
commit | 398e8c29cab05e3d06a73bc9fae3a4ad428c1095 (patch) | |
tree | f199eb5184545cb8b0c44ca711691ad2bfe4d3da /api/src/routes/channels/#channel_id | |
parent | fix #295 (diff) | |
download | server-398e8c29cab05e3d06a73bc9fae3a4ad428c1095.tar.xz |
:sparkles: use DiscordApiErrors and check limits
Diffstat (limited to 'api/src/routes/channels/#channel_id')
-rw-r--r-- | api/src/routes/channels/#channel_id/pins.ts | 3 | ||||
-rw-r--r-- | api/src/routes/channels/#channel_id/webhooks.ts | 9 |
2 files changed, 9 insertions, 3 deletions
diff --git a/api/src/routes/channels/#channel_id/pins.ts b/api/src/routes/channels/#channel_id/pins.ts index 96a3fdbf..fafb789f 100644 --- a/api/src/routes/channels/#channel_id/pins.ts +++ b/api/src/routes/channels/#channel_id/pins.ts @@ -1,6 +1,7 @@ import { Channel, ChannelPinsUpdateEvent, Config, emitEvent, getPermission, Message, MessageUpdateEvent } from "@fosscord/util"; import { Router, Request, Response } from "express"; import { HTTPError } from "lambert-server"; +import { DiscordApiErrors } from "../../../util/Constants"; const router: Router = Router(); @@ -16,7 +17,7 @@ router.put("/:message_id", async (req: Request, res: Response) => { const pinned_count = await Message.count({ channel: { id: channel_id }, pinned: true }); const { maxPins } = Config.get().limits.channel; - if (pinned_count >= maxPins) throw new HTTPError("Max pin count reached: " + maxPins); + if (pinned_count >= maxPins) throw DiscordApiErrors.MAXIMUM_PINS.withParams(maxPins); await Promise.all([ Message.update({ id: message_id }, { pinned: true }), diff --git a/api/src/routes/channels/#channel_id/webhooks.ts b/api/src/routes/channels/#channel_id/webhooks.ts index 775053ba..d2a4b9a0 100644 --- a/api/src/routes/channels/#channel_id/webhooks.ts +++ b/api/src/routes/channels/#channel_id/webhooks.ts @@ -1,11 +1,12 @@ import { Router, Response, Request } from "express"; import { check, Length } from "../../../util/instanceOf"; -import { Channel, getPermission, trimSpecial } from "@fosscord/util"; +import { Channel, Config, getPermission, trimSpecial, Webhook } from "@fosscord/util"; import { HTTPError } from "lambert-server"; import { isTextChannel } from "./messages/index"; +import { DiscordApiErrors } from "../../../util/Constants"; const router: Router = Router(); -// TODO: +// TODO: webhooks // TODO: use Image Data Type for avatar instead of String router.post("/", check({ name: new Length(String, 1, 80), $avatar: String }), async (req: Request, res: Response) => { @@ -15,6 +16,10 @@ router.post("/", check({ name: new Length(String, 1, 80), $avatar: String }), as isTextChannel(channel.type); if (!channel.guild_id) throw new HTTPError("Not a guild channel", 400); + const webhook_count = await Webhook.count({ channel_id }); + const { maxWebhooks } = Config.get().limits.channel; + if (webhook_count > maxWebhooks) throw DiscordApiErrors.MAXIMUM_WEBHOOKS.withParams(maxWebhooks); + const permission = await getPermission(req.user_id, channel.guild_id); permission.hasThrow("MANAGE_WEBHOOKS"); |