search

stevenmaguire / oauth2-keycloak

Keycloak Provider for OAuth 2.0 Client
MIT License
204 stars 151 forks source link

Fix default scopes for KC20 #56

Closed micbis closed 1 year ago

micbis commented 1 year ago

Based on issue #55, added the required scope openid to the default list.

micbis commented 1 year ago

Hi @stevenmaguire, @mstefan21

Any chance this gets merged and a new release?

Best regards Michael

mstefan21 commented 1 year ago

@micbis I will look at Wednesday night

micbis commented 1 year ago

Hi @mstefan21

Great, thanks for checking.

Best regards Michael

mstefan21 commented 1 year ago

@micbis please fix code with coding-styles and errors from failed checks and rebase on last commit. thank you

micbis commented 1 year ago

Hi @mstefan21

The pull request was already based on the last commit. I also fixed the code style error, which was unrelated to this change.

Best regards Michael

micbis commented 1 year ago

Hi @mstefan21 @stevenmaguire

Is there anything else we can help with? It would be great if this could be merged. Also see our other suggestion (issue #58).

Best regards Michael

mstefan21 commented 1 year ago

@micbis I am working now on release of new version. I made your changes in my branch with implementing your idea with add version.

mstefan21 commented 1 year ago

Released as new version 3.2.0