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.

Sign up for GitHub

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

Jump to bottom

Banned emails tld blocking #5440

Closed
arraypress opened this issue Feb 5, 2017 · 6 comments
Closed

Banned emails tld blocking #5440

arraypress opened this issue Feb 5, 2017 · 6 comments
Labels
priority-low type-feature
Milestone
2.8

Comments

@arraypress
Copy link
Contributor

Having the ability to block troublesome tlds (such as ones associated with spam and fraud) would in some instances be a lot more useful than blocking domains and emails.
@arraypress
Copy link
Contributor Author

arraypress commented Feb 5, 2017 •
edited

The individual ltds could each be on a new row with a .tld style. Multi-part tlds like .co.uk and alike would need to be taken into account also.

@pippinsplugins
Copy link
Contributor

This is a nice idea. I like it.

JeroenSormani added a commit to JeroenSormani/Easy-Digital-Downloads that referenced this issue Mar 24, 2017
@JeroenSormani
Fix awesomemotive#5440 - Allow TLDs to be added to the 'banned emails…
b08c3bf 
…' list
JeroenSormani added a commit to JeroenSormani/Easy-Digital-Downloads that referenced this issue Mar 24, 2017
@JeroenSormani
Fix awesomemotive#5440 - Allow TLDs to be added to the 'banned emails…
d94a5cc 
…' list
JeroenSormani added a commit to JeroenSormani/Easy-Digital-Downloads that referenced this issue Mar 24, 2017
@JeroenSormani
Fix awesomemotive#5440 - Allow TLDs to be added to the 'banned emails…
9da7c39 
…' list
@JeroenSormani JeroenSormani mentioned this issue Mar 24, 2017
Closed
@JeroenSormani
Copy link
Contributor

Created a PR for this here: #5440

@pippinsplugins pippinsplugins added this to the 2.8 milestone Apr 3, 2017
@pippinsplugins pippinsplugins mentioned this issue Jun 7, 2017
Merged
pippinsplugins added a commit that referenced this issue Jun 7, 2017
@pippinsplugins
Fix minor formatting. #5440
1d8b503
@pippinsplugins
Copy link
Contributor

This works great for me. Once the unit tests pass, I'll merge it in.

pippinsplugins added a commit that referenced this issue Jun 7, 2017
@pippinsplugins
Make email checks case insensitive. #5440
4d8b5ef
pippinsplugins added a commit that referenced this issue Jun 7, 2017
@pippinsplugins
Add more unit tests for banned emails. #5440
a8d70ff
cklosowski pushed a commit that referenced this issue Jun 7, 2017
@pippinsplugins @cklosowski
Issue/5440 (#5792)
9a02e76 
* Fix #5440 - Allow TLDs to be added to the 'banned emails' list

* Fix minor formatting. #5440

* Make email checks case insensitive. #5440

* Add more unit tests for banned emails. #5440
@SeanTOSCD
Copy link
Contributor

It's working but should something like .co block .co.uk as well? It's not at the moment. If that's intentional, everything's good on my end.

@pippinsplugins
Copy link
Contributor

pippinsplugins commented Jun 7, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority-low type-feature
Projects
None yet
Milestone
2.8
Development

No branches or pull requests
6 participants
@pippinsplugins @cklosowski @sunnyratilal @SeanTOSCD @JeroenSormani @arraypress