"date" header is not reliable

[perk11]

Sometimes date header is not in a proper format, for example, mail.ru returns this for date header: =?utf-8?B?0fAsIDQg4OLjIDIwMTAgMTM6MDE6NDAgKzAzMDA=?=
This makes library fail in file Header.php on line 33. Using maildate from headers array instead solved it for me, but I'm not sure if this is a right way to do this.

[ddeboer]

Has this been fixed in 49b62f0?

[KundKMC]

Hey guys, no it has not been fixed becaus I ran into the same issue today using the latest release. $message->getDate() returned null because ONLY maildate is set

imap/src/Message/AbstractMessage.php

Line 114 in 368bae1

$dateHeader = $this->getHeaders()->get('date');

[Slamdunk]

Can you post here the original raw message that gets you this behavior please?

[Slamdunk]

Well in this email the Date header misses completely, there is no maildate header neither.

How could your fix KundKMC@2a7fb4a work?

[KundKMC]

You are right, there is not Header Field. But this code gets the job done:

try {
                $date = $message->getDate();
            } catch (\Throwable $ex) {
                die("FAIL:(");
            }            
if ($date == null) {
                $alteredValue = $message->getHeaders()->get('maildate');
                $alteredValue = \str_replace(',', '', $alteredValue);
                $alteredValue = \preg_replace('/^[a-zA-Z]+ ?/', '', $alteredValue);
                $alteredValue = \preg_replace('/ +\(.*\)/', '', $alteredValue);
                $alteredValue = \preg_replace('/\bUT\b/', 'UTC', $alteredValue);
                if (0 === \preg_match('/\d\d:\d\d:\d\d.* [\+\-]\d\d:?\d\d/', $alteredValue)) {
                    $alteredValue .= ' +0000';
                }
                try {
                    $date = new \DateTimeImmutable($alteredValue);
                } catch (\Throwable $ex) {
                    throw new InvalidDateHeaderException(\sprintf('Invalid Date header found: "%s"', $dateHeader), 0, $ex);
                }
            }

$date is 2013-06-19 14:49:55 with that E-Mail.
Maybe the field is set by the so called "fix" from 2015?

[Slamdunk]

The only possibility for your snippet to work is the presence of a Maildate header.

I can only help with the original raw message 😞

[KundKMC]

Okay. I put some offort in it and it seems to work both ways:

mail.eml.gz

$mbox = "TESTCASE" . time();
$imap->createMailbox($mbox);
$mailbox = $imap->getMailbox($mbox);
$mailbox->addMessage(file_get_contents("/tmp/mail.eml"));
$messages = $mailbox->getMessages();
foreach ($messages as $message) {
    try {
        $date = $message->getDate();
    } catch (\Throwable $ex) {
        $date = new DateTimeImmutable("2009-01-02T00:00:00+0100");
    }
    if ($date == null) {
        var_dump($message->getHeaders()->get('maildate'));
    }
}

Creates a mailbox, uploads the EML File and shows:

string(26) "29-Nov-2018 15:46:40 +0100"

WTF!? It seems that there is some ReWriting going on, even while creating a message.

[Slamdunk]

Yup, it seems there an overwrite. It is likely that it happens only when querying the server through IMAp protocol, and the email remains untouched.