Laravel try catch not working, cant catch exception from a package

Question

I am using a package https://github.com/barbushin/php-imap to read email from mail server, I have the following code

$folder = Storage::disk('local')->getAdapter()->getPathPrefix();
try {
    $mailbox = new \PhpImap\Mailbox('{imap.gmail.com:993/imap/ssl}INBOX', 'xxxxx@gmail.com', 'xxxxxxxxxx', $folder);    
}
catch(\Exception $e) {
    return 'rroor';
}

but is not catching error, I want to log the error if login fails.

The following code is throwing the exception

if(!$result) {
    $errors = imap_errors();
    if($errors) {
        if($throwExceptionClass) {
            throw new $throwExceptionClass("IMAP method imap_$methodShortName() failed with error: " . implode('. ', $errors));
        }
        else {
            return false;
        }
    }
}

How can I catch this exception on my controller method ?

see the error page

score 4 · Accepted Answer · answered Aug 03 '18 at 08:53

You only have the class constructor in the try...catch. I looked at the repo and it doesn't appear to throw that exception from the constructor. https://github.com/barbushin/php-imap/blob/master/src/PhpImap/Mailbox.php#L33

Is there more to your code that may be calling this part of the code https://github.com/barbushin/php-imap/blob/master/src/PhpImap/Mailbox.php#L148?

I think you need to wrap more of your code into the try...catch that's missing from your example.

Arnab Rahman · Answer 2 · 2018-08-03T08:50:45.653

2

try {
    $folder = Storage::disk('local')->getAdapter()->getPathPrefix();
    $mailbox = new \PhpImap\Mailbox('{imap.gmail.com:993/imap/ssl}INBOX', 
   'xxxxx@gmail.com', 'xxxxxxxxxx', $folder);    
}
catch(\Throwable $e) {
   return 'error';
}

edited Aug 03 '18 at 08:50

answered Aug 03 '18 at 08:45

Arnab Rahman

1,003
1
9
15

J. Doe · Answer 3 · 2018-08-03T08:56:44.527

1

Try this:

use PhpImap\ConnectionException;

//....

$folder = Storage::disk('local')->getAdapter()->getPathPrefix();
try {
    $mailbox = new \PhpImap\Mailbox('{imap.gmail.com:993/imap/ssl}INBOX', 'xxxxx@gmail.com', 'xxxxxxxxxx', $folder);    
}
catch(ConnectionException $e) {
    return 'rroor';
}

Or edit your App\Exceptions\Handler;

public function render($request, Exception $exception)
{
    if ($exception instanceof \PhpImap\ConnectionException\ConnectionException) {
       //add log
       return 'error';
    }
    return parent::render($request, $exception);
}

edited Aug 03 '18 at 08:56

answered Aug 03 '18 at 08:44

J. Doe

1,682
11
22

`instanceof \PhpImap\ConnectionException` – lagbox Aug 03 '18 at 08:53

Laravel try catch not working, cant catch exception from a package

3 Answers3