Headers: no catchable exception

[wujku]

When date is incorrect, I can not get any of message header.

imap/src/Message/Headers.php

Line 65 in 0788d5d

throw new InvalidDateHeaderException(\sprintf('Invalid Date header found: "%s"', $value), 0, $ex);

I suggest to parse date header when user use getDate() method or return false instead of InvalidDateHeaderException.

try {
    $message->getId();
} catch (InvalidDateHeaderException $e) {
    ???
}

[Slamdunk]

I'll look at this, hoping to push a fix without a BC break.

In the meantime, can you post please the Date header that produced the exception?

[Slamdunk]

Lazy loading the parameters is a good idea, unfortunately Headers extends ArrayIterator and lazy loading an item involves too much code overwrite, because a user can access to a header through a lot of methods (current, getArrayCopy, next, offsetGet, get, seek, serialize etc).

I'm going to tag this for v2.0, but I'm going to release a major version not before 6 months from now.

[wujku]

This is a raw email which cause this error.
I fetch messages from imap account, which I used to phpunit tests.

Date: Thu, 05 Oct 2017 13:14:32 +0000
From: [email protected]
To: [email protected]
Subject: Message 2
MIME-Version: 1.0
Content-Type: text/plain
Content-Transfer-Encoding: 8bit

Message 259d6303839b75

[Slamdunk]

Thu, 05 Oct 2017 13:14:32 +0000 doesn't raise an Exception, it is correctly parsed to 2017-10-05T13:14:32+0000

[wujku]

@Slamdunk Sorry, this mail has below content:

Subject: test
MIME-Version: 1.0
Content-Type: text/plain
Date: Fri!
From: [email protected]
To: [email protected]

Hi!

[wujku]

@Slamdunk Do you have idea how catch this exception and have ability to access other header variables?

[Slamdunk]

At the time being, no

[Slamdunk]

Implemented in #250