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', '[email protected]', '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

Jason Grim · Accepted Answer · 2018-08-03 08:53:47Z

4

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.

answered Aug 3, 2018 at 8:53

Jason Grim

4234 silver badges7 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Arnab Rahman · Accepted Answer · 2018-08-03 08:50:45Z

2

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

edited Aug 3, 2018 at 8:50

answered Aug 3, 2018 at 8:45

Arnab Rahman

1,0421 gold badge9 silver badges15 bronze badges

Comments

J. Doe · Accepted Answer · 2018-08-03 08:56:44Z

1

Try this:

use PhpImap\ConnectionException;

//....

$folder = Storage::disk('local')->getAdapter()->getPathPrefix();
try {
    $mailbox = new \PhpImap\Mailbox('{imap.gmail.com:993/imap/ssl}INBOX', '[email protected]', '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 3, 2018 at 8:56

answered Aug 3, 2018 at 8:44

J. Doe

1,7521 gold badge14 silver badges23 bronze badges

1 Comment

lagbox Over a year ago

instanceof \PhpImap\ConnectionException

Collectives™ on Stack Overflow

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

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related