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

Email validator improvements #2059

Closed
vohmar opened this issue Jun 22, 2021 · 1 comment · Fixed by #2063
Closed

Email validator improvements #2059

vohmar opened this issue Jun 22, 2021 · 1 comment · Fixed by #2063
Assignees

Comments

@vohmar
Copy link
Contributor

vohmar commented Jun 22, 2021

  1. logging - syslog. With implemetation of retries its good to track the results straight from the db
  2. make the check level work from command line - set the validation level with command line parameter (regexp, mx, smtp)
  3. make the avoid spam feature optional - currently the requests are scattered to avoid blaclisting in smtp level requests, but its not needed for regexp and mx requests.
  4. add retries - similarly to csync, if invalid email is found it is just marked as invalid and if the status has not changed after third validation (may be configurable) run for example apply the forcedelete to the domain registrations.
  5. make sure that email verification model is take into account - so if an email address is validated on mx level it should not be retested before for example a year has passed. Then again if smtp level validation is enabled and email is validated on lower (mx or regexp) level it must be revalidated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants