email syntax not accepted

[kkretsch]

Logged in as admin I can neither send share links to that email address nor use that address as a test recipient within the admin e-mail-server section.

It seems within the file /var/www/cloud803/3rdparty/phpmailer/phpmailer/class.phpmailer.php I run into the pcre8 switch where it verifies the email with a fals result. I added some error_log lines before and after, using kai(at)kaikretschmann.de as email doesn't work there. (Replace the "at" as usual).

And I can't debug that pattern manually.

PHP 5.6.9-1~dotdeb+7.1
apache 2.2.22
debian wheezy/7.8
OwnCloud 8.0.3

[ghost]

Just posting some additional loglines from your post at #16715 (comment)

preg_match(): Compilation failed: internal error: previously-checked referenced subpattern not found at offset 728 at /var/www/cloud803/3rdparty/phpmailer/phpmailer/class.phpmailer.php#893

[DeepDiver1975]

@kkretsch can I ask you to report this issue with the upstream project? https://github.com/PHPMailer/PHPMailer/issues Thanks a lot!

[kkretsch]

As it seems here PHPMailer/PHPMailer#429 it might be a central php problem. php 5.6.8 did work and the newer ones fail. Might be some bigger issue for other web projects too.

[DeepDiver1975]

We might want to apply this workaround - PHPMailer/PHPMailer#429 (comment)

[DeepDiver1975]

THX @kkretsch for searching the upstream project!

[kkretsch]

I tried the one line patch but I still have the same problem. It first runs into the default/php switch case for mail address validation but after that enters the same pcre8 validation switch. It might get called on a second place still sithout the 'php' second argument.

[DeepDiver1975]

🤦

[kkretsch]

As I suspected: the foreign phpmailer class revalidates the receipient and sender email addresses within its own clas without any second argument defaulting to "auto" again. If I override that auto default it works for me.

[Synchro]

If it's calling the original validateAddress implementation instead of the one in your subclass, that's a side-effect of it using self::validateAddress() internally. That would be fixed properly if it used static::validateAddress() instead to have it select the correct class. Unfortunately we can't do that by default because late static binding was only introduced in PHP 5.3.

So far I think this bug is specific to PHP 5.5.25 and 5.6.9 when run through apache mod_php; when I know that for sure (probably when we see the next PHP release fix it), I'll patch around it so you don't have to deal with this silliness.

[DeepDiver1975]

I'll patch around it so you don't have to deal with this silliness.

@Synchro Thanks a lot! This is incredible! Many thanks!

[DeepDiver1975]

postponed to 8.0.6 - we are waiting for the upstream fix

[Synchro]

PHP 5.6.10 and 5.5.26 have been released and both include a new version of the PCRE lib (8.37) - can you confirm that this release doesn't suffer from this problem? The problem I have is that I can't reproduce it.

[DeepDiver1975]

@kkretsch can you retry with an updated php? THX

[kkretsch]

I just tried once more and now it works. I can send that test eMail from within the admin menu using the php transport.
Current php version was: PHP 5.6.10-1~dotdeb+7.3 (cli) (built: Jun 15 2015 07:22:09)

[DeepDiver1975]

Thanks a lot @kkretsch - closing this issue then