Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for new format of discord webhooks #11

Closed
Kwandes opened this issue Oct 17, 2021 · 1 comment
Closed

Support for new format of discord webhooks #11

Kwandes opened this issue Oct 17, 2021 · 1 comment

Comments

@Kwandes
Copy link

Kwandes commented Oct 17, 2021

Goal

Support all available discord webhook formats

Context

It seems like your code does not account for webhooks created via the discord app
When I create a webhook, its URL does not match the rules defined in the API:

// Ensure it's a Discord Webhook
if (!url?.startsWith('https://discord.com/api/webhooks/') &&
!url?.startsWith('https://ptb.discord.com/api/webhooks/') &&
!url?.startsWith('https://canary.discord.com/api/webhooks/') ) {

The webhook created via the discordApp:
https://discordapp.com/api/webhooks/1234/A1B2C3
I was able to modify the above URL to https://discord.com/api/webhooks/1234/A1B2C3 which your form has accepted, and I've been able to retrieve events from Sentry

Extra info

Discord app information:

  • Release Channel: stable,
  • Build Number: 101451
@IanMitchell
Copy link
Owner

Pretty sure discordapp.com is the old way; we've moved our domain over since then

@IanMitchell IanMitchell closed this as not planned Won't fix, can't repro, duplicate, stale Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants