Nomchy/discord.js
1
0
Fork 0
mirror of https://github.com/discordjs/discord.js.git synced 2026-03-09 08:03:30 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: Enforce limit of URLs for link buttons (#11011)

Browse Source 
feat: enforce limit of URLs for link buttons
This commit is contained in:
Jiralite
2025-07-23 15:20:56 +01:00
committed by GitHub
parent 90b3692941
commit f2fec9177f
2 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
packages/builders/__tests__/components/button.test.ts
View File

@@ -30,7 +30,7 @@ describe('Button Components', () => {
button.toJSON();
}).not.toThrowError();
expect(() => new LinkButtonBuilder().setURL('https://google.com')).not.toThrowError();
expect(() => new LinkButtonBuilder().setLabel('label').setURL('https://google.com').toJSON()).not.toThrowError();
});
test('GIVEN invalid fields THEN build does throw', () => {
@@ -61,6 +61,13 @@ describe('Button Components', () => {
expect(() => new PrimaryButtonBuilder().setCustomId('hi').setDisabled(0).toJSON()).toThrowError();
// @ts-expect-error: Invalid emoji
expect(() => new PrimaryButtonBuilder().setCustomId('hi').setEmoji('foo').toJSON()).toThrowError();
expect(() =>
new LinkButtonBuilder()
.setLabel('label')
.setURL(`https://google.com/${'a'.repeat(512)}`)
.toJSON(),
).toThrowError();
});
test('GiVEN valid input THEN valid JSON outputs are given', () => {

2
packages/builders/src/components/Assertions.ts
View File

@@ -32,7 +32,7 @@ const buttonDangerPredicate = buttonCustomIdPredicateBase.extend({ style: z.lite
const buttonLinkPredicate = buttonPredicateBase.extend({
style: z.literal(ButtonStyle.Link),
url: z.url({ protocol: /^(?:https?|discord)$/ }),
url: z.url({ protocol: /^(?:https?|discord)$/ }).max(512),
emoji: emojiPredicate.optional(),
label: labelPredicate,
});