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

Add 130k more disposable domains #196

Merged
merged 1 commit into from
Aug 20, 2022
Merged

Add 130k more disposable domains #196

merged 1 commit into from
Aug 20, 2022

Conversation

ilyazub
Copy link
Contributor

@ilyazub ilyazub commented Aug 1, 2022

Most probably, this PR will be rejected. For now, we use config/blacklisted_email_domains.yml config.

Some disposable domains are not marked as disposable. Here's the sample.

anosaterra.org
onesevensevensix.com
ibc-smp.org
culturapollensa.com
coriscouponcorner.com
maldoncastlemaine.com
cyber-systemsinc.com
alprostadil-de.info

I've merged 130k domains from the disposable repo.

Commands to update the config

$ curl -s https://raw.githubusercontent.com/disposable/disposable-email-domains/master/domains.txt >> config/disposable_email_domains.txt

$ sort config/disposable_email_domains.txt -u -o config/disposable_email_domains_uniq.txt

$ mv config/disposable_email_domains_uniq.txt config/disposable_email_domains.txt

Most probably, this commit won't be included to the upstream repo.
They seem to [not accept changes to the blacklist] and closing PRs with
updates to the list.

Commands to update the config

  curl -s
  https://raw.githubusercontent.com/disposable/disposable-email-domains/master/domains.txt
  >> config/disposable_email_domains.txt

  sort config/disposable_email_domains.txt -u -o
  config/disposable_email_domains_uniq.txt

  mv config/disposable_email_domains_uniq.txt
  config/disposable_email_domains.txt

[1]: #34 (comment)
@micke
Copy link
Owner

micke commented Aug 20, 2022

Hello Ilya.
I hope you are safe.

So sorry for not attending to your PR earlier.
I really appreciate your contribution, i had no idea about the project you pulled these providers from.

🇺🇦 Slava Ukraini! 🇺🇦

@micke micke merged commit 47e79d8 into micke:master Aug 20, 2022
@micke
Copy link
Owner

micke commented Aug 20, 2022

I've released version 4.0.4 to rubygems!

@ilyazub ilyazub deleted the feat/100k-more-disposable-domains branch August 26, 2022 14:10
@ilyazub
Copy link
Contributor Author

ilyazub commented Aug 26, 2022

Thanks, @micke! I really haven't expected you'll merge this PR due to the number of domains being added.

Thank you ♥️

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