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

Don't include example.com in disposable_emails.yml #59

Closed
kenn opened this issue Dec 16, 2016 · 5 comments
Closed

Don't include example.com in disposable_emails.yml #59

kenn opened this issue Dec 16, 2016 · 5 comments

Comments

@kenn
Copy link

kenn commented Dec 16, 2016

example.com is a special reserved domain defined in RFC 2606 that no provider can take.

  • It's not "disposable" - it's just never unreachable. I think the definition of "disposable" should be "email address that is reachable," not a random unreachable addresses. Otherwise disposable_emails.yml has to include any random string like fasdfatyiehjdnbsdfasd.com which is valid in syntax but also unreachable in the same sense as example.com.
  • Many of us use example.com domain for test fixtures / factories (because that's what it's intended for!), and all tests start to fail when we introduce this gem with disposable: true
@micke micke closed this as completed in 32c78cb Dec 16, 2016
@micke
Copy link
Owner

micke commented Dec 16, 2016

Good spot, thanks!

I removed example.com and release v1.2.9.

@kenn
Copy link
Author

kenn commented Nov 17, 2017

This is back as of 2.1.0 - example.com is added back on line 1152 in disposable_emails.yml.

Should I add a test so it will never happen again?

@kenn
Copy link
Author

kenn commented Nov 17, 2017

Or, override with whitelist config?

@kenn
Copy link
Author

kenn commented Nov 17, 2017

At this moment, I've monkey-patched with the following line in an initializer:

ValidEmail2.disposable_emails.delete('example.com')

@micke
Copy link
Owner

micke commented Nov 30, 2017

@kenn Sorry for the late reply.
I've removed example.com again and added a test so that it can't' sneak itself in again.
Update to version 2.1.1 and you can remove your monkey-patch
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

No branches or pull requests

2 participants