added easier method to customize jwt token payload

[svycka]

as suggested in #793
@bshaffer is it something like this?

usage example:

class MyJwtToken extends \OAuth2\ResponseType\JwtAccessToken
{
    protected function createPayload($client_id, $user_id, $scope = null)
    {
        $payload = parent::createPayload($client_id, $user_id, $scope);
        $payload['custom_param'] = 'my important value';

        return $payload;
    }
}


$server = new OAuth2\Server($storage, $config);

$jwtTokenResponseType = new MyJwtToken($publicKeyStorage, $accessTokenStorage, $refreshTokenStorage, $config);
$server->addResponseType($jwtTokenResponseType, 'token');

[bshaffer]

This is great! I'm being picky here, but I don't think the term generate is accurate for this case. I'd prefer the method be createPayload. Can you make this change?

[svycka]

@bshaffer done. I always have problems choosing names :) thought if we have generateAccessToken() generateRefreshToken() then why not generatePayload() :D not sure why those different. But anyway I did the change is it ok now? Also not sure but I guess tests are not required in this case?

[bshaffer]

@svycka I don't think tests are necessary for such a small refactoring.

As for the name, with the previous examples, the "generate" refers to the randomization of the token ID. In this case, the method is just constructing the payload and not really generating anything. That's my reasoning.

[bshaffer]

Thank you for the PR!

[svycka]

@bshaffer is it possible that this will be merged soon? or should I go with workaround for now?

[ElectroLutz]

Is there an estimated date when this will be in a release? I find it very useful.

[MajesticMaje]

Any word on this? Would be a big help!

[svycka]

because it is merged an will be released in new version.

…

On Sep 7, 2017 6:37 PM, "bluebaroncanada" ***@***.***> wrote: @svycka <https://github.com/svycka>, can you advise us why you closed this?

[bluebaroncanada]

Yeah. Sorry. I thought you were saying, "Closed because it's not easy to do."

[bluebaroncanada]

@MajesticMaje so if you use the develop branch, you should be all set.

[bshaffer]

@MajesticMaje @svycka @bluebaroncanada Sorry for the delay here, we're all tagged up! Please update your dependencies for ^1.10:

composer require bshaffer/oauth2-server-php:^1.10