Skip to content

Mailify#1985

Merged
thomas-zahner merged 7 commits intolycheeverse:masterfrom
thomas-zahner:mailify
Feb 3, 2026
Merged

Mailify#1985
thomas-zahner merged 7 commits intolycheeverse:masterfrom
thomas-zahner:mailify

Conversation

@thomas-zahner
Copy link
Member

@thomas-zahner thomas-zahner commented Jan 11, 2026 •
edited
Loading

Replace check-if-email-exists with mailify.

Rationale

Old example run

This is master with check-if-email-exists. The results seem to be quite wrong or useless.

echo 'foobar@gmail.com asdf@gmail.com foobar@gmx.de fail@example.com' | cargo run --all-features - --include-mail -v

     [200] mailto:fail@example.com
     [200] mailto:foobar@gmx.de
     [200] mailto:asdf@gmail.com
     [200] mailto:foobar@gmail.com

🔍 4 Total (in 0s) ✅ 4 OK 🚫 0 Errors

New example run

With this PR we actually get useful and well categorised results. Note that that running this on different networks/machines might yield more Unknown results. See State of SMTP for the technical reasons.

echo 'foobar@gmail.com asdf@gmail.com foobar@gmx.de fail@example.com' | cargo run --all-features - --include-mail -v

   [ERROR] mailto:fail@example.com | Unreachable mail address mailto:fail@example.com: No MX records found for domain
 [UNKNOWN] mailto:foobar@gmx.de | Unknown mail status: Mail server expected our origin IP to have a DNS PTR for reverse lookup
   [ERROR] mailto:asdf@gmail.com | Unreachable mail address mailto:asdf@gmail.com: Mail server rejects the address
     [200] mailto:foobar@gmail.com
Issues found in 1 input. Find details below.

[stdin]:
   [ERROR] mailto:asdf@gmail.com | Unreachable mail address mailto:asdf@gmail.com: Mail server rejects the address
   [ERROR] mailto:fail@example.com | Unreachable mail address mailto:fail@example.com: No MX records found for domain

🔍 4 Total (in 0s) ✅ 1 OK 🚫 2 Errors ❓ 1 Unknown

@thomas-zahner thomas-zahner force-pushed the mailify branch 2 times, most recently from 121628f to 23865b4 Compare January 12, 2026 10:16
@thomas-zahner thomas-zahner requested a review from mre January 12, 2026 10:21
@mre mre added the triage label Jan 14, 2026
@thomas-zahner thomas-zahner self-assigned this Jan 27, 2026
@thomas-zahner thomas-zahner merged commit 278c6c0 into lycheeverse:master Feb 3, 2026
7 checks passed
@thomas-zahner thomas-zahner mentioned this pull request Feb 3, 2026
Closed
@mre mre mentioned this pull request Feb 2, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mre mre Awaiting requested review from mre

Assignees

@thomas-zahner thomas-zahner

Labels

triage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Licensing issue: drop check-if-email-exists dependency

2 participants

@thomas-zahner @mre