search

klapaudius / FOSOAuthServerBundle

A server side OAuth2 Bundle for Symfony 5.0 or higher
7 stars 22 forks source link

Use user username instead of client identifier (which is a random id) #14

Closed victormacko closed 1 year ago

victormacko commented 1 year ago

Feel free to enlighten me, but using the client id uses a random number for the username, vs the actual user's username.

This also addresses the problem of if a custom ClientManager is used (ie. a static list/external service/etc), and the $accessToken->getClient() doesn't actually reference any client, then an error is thrown indicating the $client var within $accessToken needs to be set before it can be accessed (php8 error).

Thanks for reviewing my PR - nice job on getting the project working with Symfony6.