Bugfix Log RFC 2822 date parse error for #1554

the-djmaze · the-djmaze · commit 6cafd65690ff · 2024-04-30T02:58:22.000+02:00
diff --git a/snappymail/v/0.0.0/app/libraries/MailSo/Base/DateTimeHelper.php b/snappymail/v/0.0.0/app/libraries/MailSo/Base/DateTimeHelper.php
@@ -37,15 +37,15 @@ public static function ParseRFC2822DateString(string $sDateTime) : int
 	{
 		$sDateTime = \trim($sDateTime);
 		if (empty($sDateTime)) {
-			\SnappyMail\Log::info("No RFC 2822 date to parse");
+			\SnappyMail\Log::info('', "No RFC 2822 date to parse");
 			return 0;
 		}
 
 		$sDateTime = \trim(\preg_replace('/ \([a-zA-Z0-9]+\)$/', '', $sDateTime));
 		$oDateTime = \DateTime::createFromFormat(\DateTime::RFC2822, $sDateTime, static::GetUtcTimeZoneObject());
 		// 398045302 is 1982-08-13 00:08:22 the date RFC 822 was created
 		if (!$oDateTime || 398045302 > $oDateTime->getTimestamp()) {
-			\SnappyMail\Log::notice("Failed to parse RFC 2822 date '{$sDateTime}'");
+			\SnappyMail\Log::notice('', "Failed to parse RFC 2822 date '{$sDateTime}'");
 		}
 		return $oDateTime ? $oDateTime->getTimestamp() : 0;
 	}

Original file line number	Diff line number	Diff line change
`@@ -37,15 +37,15 @@ public static function ParseRFC2822DateString(string $sDateTime) : int`
`37`	`37`	`{`
`38`	`38`	`$sDateTime = \trim($sDateTime);`
`39`	`39`	`if (empty($sDateTime)) {`
`40`		`- \SnappyMail\Log::info("No RFC 2822 date to parse");`
	`40`	`+ \SnappyMail\Log::info('', "No RFC 2822 date to parse");`
`41`	`41`	`return 0;`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`$sDateTime = \trim(\preg_replace('/ \([a-zA-Z0-9]+\)$/', '', $sDateTime));`
`45`	`45`	`$oDateTime = \DateTime::createFromFormat(\DateTime::RFC2822, $sDateTime, static::GetUtcTimeZoneObject());`
`46`	`46`	`// 398045302 is 1982-08-13 00:08:22 the date RFC 822 was created`
`47`	`47`	`if (!$oDateTime \|\| 398045302 > $oDateTime->getTimestamp()) {`
`48`		`- \SnappyMail\Log::notice("Failed to parse RFC 2822 date '{$sDateTime}'");`
	`48`	`+ \SnappyMail\Log::notice('', "Failed to parse RFC 2822 date '{$sDateTime}'");`
`49`	`49`	`}`
`50`	`50`	`return $oDateTime ? $oDateTime->getTimestamp() : 0;`
`51`	`51`	`}`