Skip to content

fix(security): validate problem source hosts#93

Merged
cameroncuster merged 1 commit into
mainfrom
cameroncuster/fix-problem-source-host-validation
Jul 23, 2026
Merged

fix(security): validate problem source hosts#93
cameroncuster merged 1 commit into
mainfrom
cameroncuster/fix-problem-source-host-validation

Conversation

@cameroncuster

Copy link
Copy Markdown
Owner

Summary

  • replace Kattis hostname substring classification with the existing exact-host URL validator
  • move source classification into a dependency-free helper shared by application and E2E fixture logic
  • add adversarial regression cases for subdomain, path, userinfo, query, and malformed-string tricks

Security

Resolves the three CodeQL js/incomplete-url-substring-sanitization findings by using parsed, exact-match Kattis hosts rather than url.includes('kattis.com').

Verification

  • Prettier passed
  • ESLint passed
  • Svelte check passed
  • unit tests: 199/199 passed
  • targeted desktop/mobile data E2E: 14/14 passed
  • git diff --check passed
  • independent security review blocker resolved

Pull Request opened by Augment Code | View session

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gitgud Ready Ready Preview, Comment Jul 23, 2026 9:32pm

@cameroncuster
cameroncuster marked this pull request as ready for review July 23, 2026 21:32
@cameroncuster
cameroncuster merged commit b7da916 into main Jul 23, 2026
6 checks passed
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.

1 participant