mirror/php-e107
1
0
Fork 0
mirror of https://github.com/e107inc/e107.git synced 2025-08-05 14:17:49 +02:00
Code Issues Releases Wiki Activity

Fix for embedded image data causing mailing failure after upgrading PHPMailer.

Browse Source
This commit is contained in:
Cameron
2020-03-04 18:13:05 -08:00
parent a3d99f0d19
commit bb6fe3283b
1 changed files with 40 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
e107_handlers/mail.php
View File

@@ -433,7 +433,7 @@ class e107Email extends PHPMailer
{ {
if ($this->logEnable && ($this->logHandle === false)) if ($this->logEnable && ($this->logHandle === false))
{ {
$logFileName = MAIL_LOG_PATH.'mailoutlog.txt'; $logFileName = MAIL_LOG_PATH.'mailoutlog.log';
$this->logHandle = fopen($logFileName, 'a'); // Always append to file $this->logHandle = fopen($logFileName, 'a'); // Always append to file
} }
if ($this->logHandle !== false) if ($this->logHandle !== false)
@@ -534,7 +534,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
break; break;
@@ -545,7 +545,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
break; break;
@@ -562,7 +562,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
} }
@@ -580,7 +580,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
} }
@@ -708,7 +708,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
} }
@@ -746,7 +746,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// do something $this->logLine($e->getMessage());
} }
@@ -1093,17 +1093,26 @@ class e107Email extends PHPMailer
$this->SendCount++; $this->SendCount++;
if($this->isError())
{
$this->logLine('Error info: '.$this->ErrorInfo);
e107::getMessage()->addInfo($this->ErrorInfo);
}
$result = false; $result = false;
if ($this->debug == false && (($this->logEnable == 0) || ($this->logEnable == 2)) ) if ($this->debug == false && (($this->logEnable == 0) || ($this->logEnable == 2)) )
{ {
// prevent user/script details being exposed in X-PHP-Script header // prevent user/script details being exposed in X-PHP-Script header
if(!empty($_SERVER['REMOTE_ADDR']))
{
$oldphpself = $_SERVER['PHP_SELF']; $oldphpself = $_SERVER['PHP_SELF'];
$oldremoteaddr = $_SERVER['REMOTE_ADDR']; $oldremoteaddr = $_SERVER['REMOTE_ADDR'];
$_SERVER['PHP_SELF'] = "/"; $_SERVER['PHP_SELF'] = "/";
$_SERVER['REMOTE_ADDR'] = $_SERVER['SERVER_ADDR']; $_SERVER['REMOTE_ADDR'] = $_SERVER['SERVER_ADDR'];
$_SERVER["HTTP_X_FORWARDED_FOR"] = $_SERVER['SERVER_ADDR']; $_SERVER["HTTP_X_FORWARDED_FOR"] = $_SERVER['SERVER_ADDR'];
$_SERVER["HTTP_CF_CONNECTING_IP"] = $_SERVER['SERVER_ADDR']; $_SERVER["HTTP_CF_CONNECTING_IP"] = $_SERVER['SERVER_ADDR'];
}
try try
{ {
@@ -1111,15 +1120,16 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
// echo "Message could not be sent. Mailer Error: {$mail->ErrorInfo}"; $this->logLine($e->getMessage());
} }
if(!empty($oldremoteaddr))
{
$_SERVER['PHP_SELF'] = $oldphpself; $_SERVER['PHP_SELF'] = $oldphpself;
$_SERVER['REMOTE_ADDR'] = $oldremoteaddr; $_SERVER['REMOTE_ADDR'] = $oldremoteaddr;
$_SERVER["HTTP_X_FORWARDED_FOR"] = $oldremoteaddr; $_SERVER["HTTP_X_FORWARDED_FOR"] = $oldremoteaddr;
$_SERVER["HTTP_CF_CONNECTING_IP"] = $oldremoteaddr; $_SERVER["HTTP_CF_CONNECTING_IP"] = $oldremoteaddr;
}
if (!$bulkmail && !$this->SMTPKeepAlive && ($this->Mailer == 'smtp')) $this->smtpClose(); if (!$bulkmail && !$this->SMTPKeepAlive && ($this->Mailer == 'smtp')) $this->smtpClose();
} }
@@ -1231,7 +1241,10 @@ class e107Email extends PHPMailer
foreach($images[3] as $i => $url) foreach($images[3] as $i => $url)
{ {
if(strpos($url,'data:') === 0) // already embedded, so skip processing
{
continue;
}
// do not change urls for absolute images (thanks to corvuscorax) // do not change urls for absolute images (thanks to corvuscorax)
if (!preg_match('#^[A-z]+://#',$url)) if (!preg_match('#^[A-z]+://#',$url))
{ {
@@ -1289,6 +1302,7 @@ class e107Email extends PHPMailer
} }
catch (Exception $e) catch (Exception $e)
{ {
$this->logLine($e->getMessage());
if ($this->debug) if ($this->debug)
{ {
echo "Add embedded image {$url} failed<br />"; echo "Add embedded image {$url} failed<br />";
@@ -1339,7 +1353,7 @@ class e107Email extends PHPMailer
$this->AltBody = 'To view this email message, enable HTML!' . "\n\n"; $this->AltBody = 'To view this email message, enable HTML!' . "\n\n";
} }
return null; return $this->Body;
} }
@@ -1384,7 +1398,7 @@ class e107Exception extends Exception
{ {
parent::__construct($message, $code); parent::__construct($message, $code);
e107::getLog()->e_log_event(10, e107::getLog()->addEvent(10,
$this->getFile().'|@'.$this->getLine(), $this->getFile().'|@'.$this->getLine(),
'EXCEPT', 'EXCEPT',
$this->getCode().':'.$this->getMessage(), $this->getCode().':'.$this->getMessage(),