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

Fix the discord external url #792

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

evandrosaturnino
Copy link
Collaborator

@evandrosaturnino evandrosaturnino commented Dec 13, 2024

Description

Fix the Discord external url

Type of change

  • ๐Ÿ’… New Feature (Breaking/Non-breaking Change)
  • ๐Ÿ› Bug fix (Non-breaking Change: Fixes an issue)
  • ๐Ÿ› ๏ธ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..

Notice

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@@ -1,6 +1,6 @@
export enum ExternalHref {
thresholdGithub = "https://github.com/threshold-network/token-dashboard",
thresholdDiscord = "https://discord.gg/WXK9PC6SRF",
thresholdDiscord = "https://discord.gg/threshold",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discord custom invite URLs are not reserved if the server loses boost status, so they are an attack vector.

We can and may already have a discord.threshold.network or chat.threshold.network that gives us a branded URL under our own control. We've done this switch in other places including maybe some parts of the Threshold sites.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasnโ€™t aware that we had a subdomain redirecting to the Discord invite URL: https://discord.threshold.network/. This URL appears to be working correctly, so for this PR, this has been adressed on update discord url to branded subdomain

I checked, and we donโ€™t currently use this subdomain URL on the Threshold website. Iโ€™ve pushed a PR to address this as well: threshold-network/website#127.

Copy link

github-actions bot commented Jan 6, 2025

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 this pull request may close these issues.

2 participants