Suggestion to include notice for those who want to use "openid" scope in documentation.

Dhavalc2012 commented 3 years ago

Issue:

When you try to add the scope of 'openid' scope as shown on the following link (https://heimdall.ezralazuardy.com/documentation/oidc). It throws a message (error) while using token API. (http://localhost:8080/rest/token)

"Using integers for registered date claims is deprecated, please use DateTimeImmutable objects instead."

This seems to be caused by the dependency of league/oauth2-server. It uses newer package lcobucci/jwt where there have been major breaking changes. Similiar issue has been noted in laravel passport package. (https://github.com/laravel/passport/issues/1381)

Solution : This can be resolved by downgrading the package " lcobucci/jwt" to 3.3.3 for the dependency "league/oauth2-server" and then run command composer require lcobucci/jwt:3.3.3

ezralazuardy commented 3 years ago

thanks for the report. ill recheck this issue as soon as possible

ergec commented 3 years ago

https://github.com/ezralazuardy/heimdall/issues/8

This is the same issue I reported a while ago which said to be fixed in v0.0.4 but unfortunately it's not :(

ezralazuardy / heimdall

Suggestion to include notice for those who want to use "openid" scope in documentation. #13