[Proposal] Descriptive Exception Classes

[alexw23]

Is it possible to have specific exceptions so they can be caught accordingly?

At the moment I am having todo the following:

    catch (UnexpectedValueException $e)
    {
        if($e->getMessage() == "Expired Token") {
           // Logic Here
        } else {
            throw $e;
        }
    }

Instead I propose the following:

Declare an exception class (i.e. Exceptions.php)

        class ExpiredTokenException extends \UnexpectedValueException {}

Handle it in the code

        if (isset($payload->exp) && time() >= $payload->exp){
            throw new ExpiredTokenException('Expired Token');
        }

Catch It

    catch (JWT\Exceptions\ExpiredTokenException $e)
    {
        // Logic Here
    }

This can be caught much easier and would have backwards compatibility with devs using UnexpectedValueException.

[brendo]

I needed this too, so I've added it in my fork. Namespaces aren't being used because this project's composer file dictates that we should be supporting PHP 5.2. Should @drtriumph want to update the project to 2.0, we could follow PSR-4, drop PHP 5.2 support and tidy this up.

[alexw23]

Thanks @brendo will wait for the PR to close 👍

[cbraynor]

Thanks all, this was updated in #23