email_verifier icon indicating copy to clipboard operation
email_verifier copied to clipboard

Published 20 hours ago • verified publisher icon kamilc

Timeouts and a records

Open mreinsch opened this issue 9 years ago • 2 comments

This includes a couple of cleanups, reduces timeouts (30/60 secs is usually too long), and will also fallback to the A record in case no MX record is given (as per RFC).

mreinsch avatar Sep 23 '15 09:09 mreinsch

Other than the thing I commented on above, I noticed that it stopped working for me.

irb(main):002:0> EmailVerifier.check "[email protected]"
EmailVerifier::NoMailServerException: No mail server for [email protected]

While the version from master just works:

irb(main):002:0> EmailVerifier.check "[email protected]"
=> true

It can't be merged in at this moment. Could you take a look and provide any fixes? I like the change — it would be great to have it in master.

Thanks

kamilc avatar Oct 12 '15 13:10 kamilc

Thanks for reviewing it. I just tried the check you posted, and it seems to work fine for me? Could you try once more to make sure it wasn't some strange DNS issue? Otherwise I'll try to dig a bit deeper...

mreinsch avatar Oct 12 '15 13:10 mreinsch