Exception Parse Data

[GuiAndra]

Hello, I'm getting this exception

I added a try in the parse header function in the message class to pass the error.

private function parseHeader() {
$header = imap_fetchheader($this->client->connection, $this->uid, $this->fetch_options);
if ($header) {
$header = imap_rfc822_parse_headers($header);
}

if (property_exists($header, 'subject')) {
    $this->subject = imap_utf8($header->subject);
}
if (property_exists($header, 'date')) {
    try{
        $this->date = Carbon::parse($header->date);
    }catch (\Exception $e){
    }
}
if (property_exists($header, 'from')) {
    $this->from = $this->parseAddresses($header->from);
}
if (property_exists($header, 'to')) {
    $this->to = $this->parseAddresses($header->to);
}
if (property_exists($header, 'cc')) {
    $this->cc = $this->parseAddresses($header->cc);
}
if (property_exists($header, 'bcc')) {
    $this->bcc = $this->parseAddresses($header->bcc);
}

if (property_exists($header, 'reply_to')) {
    $this->reply_to = $this->parseAddresses($header->reply_to);
}
if (property_exists($header, 'sender')) {
    $this->sender = $this->parseAddresses($header->sender);
}

if (property_exists($header, 'message_id')) {
    $this->message_id = str_replace(['<', '>'], '', $header->message_id);
}
if (property_exists($header, 'Msgno')) {
    $this->message_no = ($this->fetch_options == FT_UID) ? trim($header->Msgno) : imap_msgno($this->client->getConnection(), trim($header->Msgno));
}

}

[Webklex]

Hi @GuiAndra ,
thanks for the report. I think I would rather like to validate the given datestring and perhaps "fix it" if it seems to be broken. This way anyone can be sure to always have a valid Carbon object.

Please give me a few days or perhaps the upcomming weekend to come up with a fix :)

Best regards

[GuiAndra]

Sure @Webklex, i'm waiting for the fix.

thanks :)

[Webklex]

Hi @GuiAndra,
please update to v 1.0.3.9 :)

[GuiAndra]

Thaanks @Webklex :)

[RpCoding]

04 Jan 2018 10:12:47 UT is a timestamp I have in one of the emails I'm trying to read. This caused a crash because of Carbon::parse.

Edit: My dirty fix was to replace

try{
    $this->date = Carbon::parse($date);
}catch(\Exception $e){
    switch(true){
        case preg_match('/([A-Z]{2,3}\,\ [0-9]{1,2}\ [A-Z]{2,3}\ [0-9]{4}\ [0-9]{1,2}\:[0-9]{1,2}\:[0-9]{1,2}\ \+[0-9]{4}\ \([A-Z]{2,3}\+[0-9]{1,2}\:[0-9]{1,2})\)+$/i', $date):
            $array = explode('(', $date);
            $array = array_reverse($array);
            $date = trim(array_pop($array));
            break;
    }
    $this->date = Carbon::parse($date);
}

with

try{
    $this->date = Carbon::parse($date);
}catch(\Exception $e){
    switch(true){
        case preg_match('/([A-Z]{2,3}\,\ [0-9]{1,2}\ [A-Z]{2,3}\ [0-9]{4}\ [0-9]{1,2}\:[0-9]{1,2}\:[0-9]{1,2}\ \+[0-9]{4}\ \([A-Z]{2,3}\+[0-9]{1,2}\:[0-9]{1,2})\)+$/i', $date):
            $array = explode('(', $date);
            $array = array_reverse($array);
            $date = trim(array_pop($array));
            break;
     }

    if(strpos($date, "UT") !== FALSE && strpos($date, "UTC") === FALSE){
        $date = str_replace("UT", "UTC", $date);
    }

    $this->date = Carbon::parse($date);
}

in https://github.com/Webklex/laravel-imap/blob/master/src/IMAP/Message.php but as I said, quick and dirty.

[Webklex]

@RpCoding thanks for your report :) I just released a new version containing the new validation rule.

Ref.: v. 1.0.5.5

[Davronjuraev]

Hello, please help me, How can i reply to message from laravel application