If no MX records found

[volgot]

If domain dosn't exist your script fails, hangs up and kills server
line 285
there must be "if mxQuery" ($hosts, $weights) or false (no MX servers = fake domain)
you even don't have NoMX.php in your Exceptions folder ;)
it's a good script, otherwise :)

[volgot]

plus, you would give 2 separate variables for HELO server and for USER
because, usually correct HELO looks like "mail.example.com" and user's email looks like "some_user@example.com" or even "some_user@example2.com" (not a "user@mail.example.com", it can be different IP also, different SPF, and it can lead to spam block some tester's IP). I just commented in setSender() //$parts = $this->splitEmail($email); to used hardcoded own:
$this->from_user = "real@user.com";
$this->from_domain = "real.mail.server";

[zytzagoo]

Hello and thanks for the feedback!

Line 285 is empty on current master: https://github.com/zytzagoo/smtp-validate-email/blob/master/src/Validator.php#L285 -- so I'm not really following what you're talking about.

The test suite is there, feel free to provide a test case that exhibits the behaviour you encounter (ideally maybe even a PR for a fix if it turns out something is really broken!).

Regarding customising the HELO sent, are you up for doing a PR for that too and allow it to be easily customised from the outside? (It should also preserve back-compat for existing users)

[volgot]

Helo, just try to check this address: blabla@blablabla.bla
Good luck :)

[volgot]

or, better, this one, from real life: zzmusic@zzmusic.com (caused problem)

[volgot]

p.s. sorry, I made changes in code, so 285 was a wrong line number, correct, but just find a line:
list($hosts, $weights) = $this->mxQuery($domain) (it was 275)
wich must be changed to:
// Query the MX records for the current domain
list($hosts, $weights) = $this->mxQuery($domain);

        if(empty($hosts)) {
            return array($users[0]."@".$domain=>false);
        }

as a minimum, for every address in array (I check only 1 address per time, so this easy solution works for me) :)

[volgot]

One more thing, checking several addresses at ones from one host (in your second example), maybe is not very good idéa, imho, it can lead to blacklist directly.. otherwise, is a very nice script, tnx for jour job :)

About HELO, everybody please check this:

The HELO string sent should in the style of:

Example:
HELO mta1.mycompany.com

The following bad example(s) will get rejected:
HELO
HELO localhost
HELO localhost.localdomain
HELO hostnameonly
HELO 192.168.1.1 (just an IP)
HELO [192.168.1.1]
HELO .com (starts with a period)
HELO fake..com
HELO @(&$ (characters not normally allowed in domain names)
Spammers will often be caught by this rule, when they take over a PC to act as a spam bot. They just use the hostname as the PC has it configured, which is normally not set up as a FQDN.

Source https://www.linuxmagic.com/best_practices/valid_helo_domain.html

Take a look at "hostnameonly" line..

p.s. valid HELO = mail.server.com (MX server which belongs to your email address)

Best regards

[Thanatermesis]

Im not sure if this is the same issue but I was validating an address that seems to be incorrect: something@archlabslinux.com , the domain exists but the MX does not, and seems like smtp-validate-email returns a FALSE positive for connecting to an invalid MX server (which says 0.0.0.0), see the logs:

array (
  0 => '[2021-04-14T01:34:03.948473+0000] MX records (archlabslinux.com): Array
(
    [0.0.0.0] => 1000
    [archlabslinux.com] => 0
)
',
  1 => '[2021-04-14T01:34:03.948533+0000] Connecting to 0.0.0.0:25',
  2 => '[2021-04-14T01:34:03.948692+0000] Connected to 0.0.0.0:25 successfully',
  3 => '[2021-04-14T01:34:03.956457+0000] << '[2021-04-14T01:34:03.956587+0000] send>>>: EHLO mydomain.com...
etc... ending in a valid verification