fix: Add Discord webhook URL validation before sending notifications#1803
fix: Add Discord webhook URL validation before sending notifications#1803Pratikshya32 wants to merge 23 commits into
Conversation
…ci-analytics lib with tests (Priyanshu-byte-coder#1501)
…file-data; fix compare page type errors
…blicLanguage.name TS error - Remove duplicate `aria-label="Perform action"` from 10 components (settings page, AIMentorWidget, ContributionGraph, ContributionHeatmap, GoalTracker, KeyboardShortcuts, NotificationBell, StreakAtRiskBanner, StreakMilestoneBanner, WeeklySummaryCard) - Add dompurify + @types/dompurify to deps, @testing-library/react + @testing-library/jest-dom to devDeps - Fix compare page using PublicLanguage.name → PublicLanguage.language Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
@Pratikshya32 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
This PR has merge conflicts with |
|
This PR has merge conflicts with git fetch origin
git rebase origin/main
# resolve conflicts, then:
git push --force-with-lease |
|
Closing this PR because it modifies Additionally, this PR touches 70+ unrelated files which indicates it was generated by AI without proper scoping to the issue being addressed. If you'd like to contribute the feature, please open a new PR that:
|
Summary
Adds strict URL validation for Discord webhook URLs to prevent SSRF and unexpected API errors from malformed URLs.
Changes
src/lib/validators/discordWebhook.tscc @Priyanshu-byte-coder