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

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 '[email protected] [email protected] [email protected] [email protected]' | cargo run --all-features - --include-mail -v

     [200] mailto:[email protected]
     [200] mailto:[email protected]
     [200] mailto:[email protected]
     [200] mailto:[email protected]

🔍 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 '[email protected] [email protected] [email protected] [email protected]' | cargo run --all-features - --include-mail -v

   [ERROR] mailto:[email protected] | Unreachable mail address mailto:[email protected]: No MX records found for domain
 [UNKNOWN] mailto:[email protected] | Unknown mail status: Mail server expected our origin IP to have a DNS PTR for reverse lookup
   [ERROR] mailto:[email protected] | Unreachable mail address mailto:[email protected]: Mail server rejects the address
     [200] mailto:[email protected]
Issues found in 1 input. Find details below.

[stdin]:
   [ERROR] mailto:[email protected] | Unreachable mail address mailto:[email protected]: Mail server rejects the address
   [ERROR] mailto:[email protected] | Unreachable mail address mailto:[email protected]: 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 merged commit 278c6c0 into lycheeverse:master Feb 3, 2026
7 checks passed
@mre mre mentioned this pull request Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants