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 dbmail.com #232

Merged
merged 1 commit into from
Nov 20, 2019
Merged

remove dbmail.com #232

merged 1 commit into from
Nov 20, 2019

Conversation

danmihai89
Copy link
Contributor

@danmihai89 danmihai89 commented Nov 19, 2019

"dbmail.com" is a legitimate email domain and doesn't offer disposable emails.

Please remove the blacklist, we have many customers using this domain address and this breaks our communication services.

@pvizeli
Copy link
Contributor

pvizeli commented Nov 19, 2019

Please a bit more details about the provider because the domain have website

@danmihai89
Copy link
Contributor Author

@pvizeli It seems that this is an old and now obsolete email provider. I have no other details, other than this is definitely a false positive.
It came to my attention as I observed many of our longtime customers not being processed, because they have a dbmail.com email address. This domain was added to the blacklist sometime in March 2019, but I cannot find the reason why.
A quick fix would help us a lot, because otherwise we would have to develop exceptions in our code to be able to process these email addresses.
Thanks!

@pvizeli
Copy link
Contributor

pvizeli commented Nov 20, 2019

The SMTP server looks fine, yes

@pvizeli pvizeli merged commit d0ac128 into FGRibreau:master Nov 20, 2019
@danmihai89
Copy link
Contributor Author

@pvizeli @FGRibreau do you have an ETA for a new version release with this blacklisted domain removed? I would really appreciate if you could release it this week.
PS: We use the NodeJS module in our projects.

@FGRibreau
Copy link
Owner

@danmihai89 PR were merged but tests do not pass, so I can't make a release and I'm waiting for a fix (or merge revert) from @pvizeli

@danmihai89
Copy link
Contributor Author

danmihai89 commented Nov 25, 2019

@pvizeli any updates? I really need a release for this.
Thanks!

@danmihai89
Copy link
Contributor Author

Hi, guys, @FGRibreau @pvizeli
Any updates on a release for this?
Thanks!

@FGRibreau
Copy link
Owner

Released in v3.3.3

@danmihai89
Copy link
Contributor Author

Thank you very much! Already using it in production. :)

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.

None yet

3 participants