Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove disposable domains that were removed from mailchecker #67

Merged
merged 1 commit into from
Jun 15, 2017
Merged

Remove disposable domains that were removed from mailchecker #67

merged 1 commit into from
Jun 15, 2017

Conversation

cgriego
Copy link
Contributor

@cgriego cgriego commented Jun 15, 2017

I ran your disposable domain database against our own and found many false positives for Chinese email domains. I dug and found they'd been removed from mailchecker. So I modified and ran the update script locally to not keep the existing domains, filtered to just removed domains, and then searched the mailchecker history (with git log -p -S domain -- list.json) to verify they had been removed and where.

FGRibreau/mailchecker@17af0c5
126.com
163.com
sina.com
sohu.com
vip.163.com
vip.qq.com
vip.sina.com

FGRibreau/mailchecker@fa7696d
disposableemailaddresses:emailmiser.com

FGRibreau/mailchecker@df525b9
yeah.net

FGRibreau/mailchecker@17af0c5
126.com
163.com
sina.com
sohu.com
vip.163.com
vip.qq.com
vip.sina.com

FGRibreau/mailchecker@fa7696d
disposableemailaddresses:emailmiser.com

FGRibreau/mailchecker@df525b9
yeah.net
@micke micke merged commit fc83aed into micke:master Jun 15, 2017
@micke
Copy link
Owner

micke commented Jun 15, 2017

Thanks!

@micke
Copy link
Owner

micke commented Jun 15, 2017

I've release 1.2.15 to rubygems with this PR merged.

@cgriego cgriego deleted the remove-domains-removed-from-mailchecker branch June 15, 2017 14:05
@cgriego
Copy link
Contributor Author

cgriego commented Jun 15, 2017

Thanks!

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.

2 participants