Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I had to disable the module at www.wechange.org because for every email (valid or not), we had the message:
Email host [e.g. users.sourceforge.net] does not seem valid, it does not answer
Is it possible that my host does not accept @fsockopen connections?
I am asking them, too.
Comment | File | Size | Author |
---|---|---|---|
#5 | email_verify.install.txt | 1.93 KB | beginner |
Comments
Comment #1
dbr CreditAttribution: dbr commentedI think this needs to be debugged on your host. Try printing debugging statement, turning on tracing, ...
Once the cause is found, it would be nice, if possible, to detect this situation and either warn the user or, since in doubt, accept the emails (or both).
Comment #2
beginner CreditAttribution: beginner commentedI asked my host. It seems they are set up to only accept fsockopen on port 80 and not 25. I am negotiating with them :)
Clearly the problem is on the host side, not the module.
I really don't know how we could check for this.
At the very least, we could add a word of caution, both in the README, and within the admin interface (but there is no configuration screen).
Admin would be encouraged to test with mail servers they know work (e.g. @drupal.org).
Comment #3
beginner CreditAttribution: beginner commentedWe could add a test on hook_insall() and test with @drupal.org: we know this one works.
When the module is enabled, one test is performed, and if it fails, the module is disabled again and a warning given to the admin.
?
Comment #4
dbr CreditAttribution: dbr commentedThat sounds reasonable :-)
Comment #5
beginner CreditAttribution: beginner commentedWe only need to add the attached email_verify.install.
I tested it on my local box, where the port is open,
and on my remote server, where it is not,
and I got the expected result.
See the comments in the file. I think I copied the right parts of the email_verify_check() function.
I post it here, for you to review, and if it's ok, I'll commit it tomorrow.
Comment #6
beginner CreditAttribution: beginner commentedPatch applied to HEAD and D5.
Comment #7
(not verified) CreditAttribution: commented