Skip to content

[feat]: improve email validation in backend verifier to prevent malform…#87

Open
Denisha227 wants to merge 1 commit into
sharmavaibhav31:mainfrom
Denisha227:feature/email-validation-verifier
Open

[feat]: improve email validation in backend verifier to prevent malform…#87
Denisha227 wants to merge 1 commit into
sharmavaibhav31:mainfrom
Denisha227:feature/email-validation-verifier

Conversation

@Denisha227
Copy link
Copy Markdown

@Denisha227 Denisha227 commented May 26, 2026

Overview
Previously, the backend email verifier used a very basic check (if not email or "@" not in email:) to check for valid emails. This allowed invalid or malformed email patterns (such as missing a top-level domain, local-part, or containing invalid characters) to pass through to the subsequent SMTP verification steps, wasting connection bandwidth and server resources.

This PR improves the email verification process by introducing a lightweight, robust regular expression check before SMTP verification.

Key Changes
Regex-Based Check: Added a structured regex pattern (^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,}$) in contact-discovery-service/verifier.py.
Early Rejection: Ensures malformed email formats (e.g., user@, user@domain, @domain.com, user@domain.) are instantly marked as "invalid" and rejected without triggering unnecessary rate limit checks or DNS/SMTP lookups.
Maintainability: Kept the regular expression lightweight to balance performance and coverage.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

@Denisha227 is attempting to deploy a commit to the Vaibhav Sharma's projects Team on Vercel.

A member of the Team first needs to authorize it.

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