-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Testsuite PoC - Use service account for admin client #31478
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
573c537 to
652a4b1
Compare
lhanusov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice solution with Client credentials for Admin Client -> LGTM 👍
...src/main/java/org/keycloak/quarkus/runtime/integration/jaxrs/QuarkusKeycloakApplication.java
Show resolved
Hide resolved
652a4b1 to
4c29af0
Compare
...src/main/java/org/keycloak/quarkus/runtime/integration/jaxrs/QuarkusKeycloakApplication.java
Show resolved
Hide resolved
|
@stianst Was this PR superseded? |
Nah, I deleted the branch by mistake. Re-opened now |
Signed-off-by: stianst <[email protected]>
4c29af0 to
e7ab088
Compare
|
|
||
| adminClient.realms().create(realmRepresentation); | ||
|
|
||
| // TODO Token needs to be invalidated after creating realm to have roles for new realm in the token. Maybe lightweight access tokens could help. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have an issue logged for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's for supporting it in Keycloak, will add another to update the test poc
Signed-off-by: stianst <[email protected]>
Signed-off-by: stianst [email protected]
Closes #31527