Skip to content

fix: sanitize and validate GitHub username input#112

Open
shagunkillekar5 wants to merge 2 commits into
JhaSourav07:mainfrom
shagunkillekar5:fix-github-username-validation
Open

fix: sanitize and validate GitHub username input#112
shagunkillekar5 wants to merge 2 commits into
JhaSourav07:mainfrom
shagunkillekar5:fix-github-username-validation

Conversation

@shagunkillekar5
Copy link
Copy Markdown

Description

Implemented GitHub username sanitization and validation in the landing page input field.

Changes made

  • Trimmed whitespace from usernames
  • Extracted username automatically from GitHub profile URLs
  • Added validation for invalid GitHub username characters
  • Prevented invalid submissions
  • Added user-friendly error message for invalid usernames

Tested

  • Valid usernames work correctly
  • Invalid usernames show error
  • GitHub URLs are sanitized properly

Fixes #63

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 16, 2026

@shagunkillekar5 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@JhaSourav07
Copy link
Copy Markdown
Owner

CI is failing can you fix it and also PR description should be as template

@JhaSourav07 JhaSourav07 added status: blocked PRs with merge conflicts, failing builds, status: hold labels May 17, 2026
@github-actions github-actions Bot added the needs-details This PR is missing required description details. label May 17, 2026
@github-actions
Copy link
Copy Markdown

👋 Hey @shagunkillekar5, it looks like you didn't use our PR template!

The section ## Pillar is missing from your PR description.

Please update your PR description to include all required sections so we can review this properly:

  • ## Description — What does this PR do? Which issue does it fix?
  • ## Pillar — Which contribution pillar does this fall under?
  • ## Checklist — Have you ticked off the quality checklist?

You can find the full template in CONTRIBUTING.md. Just edit your PR description and the needs-details label will be removed automatically. 🙌

@github-actions github-actions Bot added the needs-rebase This PR has merge conflicts and needs a rebase. label May 17, 2026
@github-actions
Copy link
Copy Markdown

⚠️ Hey @shagunkillekar5, this PR has merge conflicts with the main branch.

Please pull the latest changes and resolve the conflicts so we can review it!

git fetch origin
git rebase origin/main
# resolve any conflicts, then:
git push --force-with-lease

Once resolved, the needs-rebase label will be removed automatically on the next check. 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-details This PR is missing required description details. needs-rebase This PR has merge conflicts and needs a rebase. status: blocked PRs with merge conflicts, failing builds, status: hold

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Security & UX: Sanitize GitHub username input before submission

2 participants