Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@stianst
Copy link
Contributor

@stianst stianst commented Jul 22, 2024

Signed-off-by: stianst [email protected]

Closes #31527

@stianst stianst requested a review from a team as a code owner July 22, 2024 11:49
@stianst stianst force-pushed the testsuite-poc-service-account branch from 573c537 to 652a4b1 Compare July 22, 2024 11:52
@stianst stianst requested a review from a team as a code owner July 22, 2024 11:52
Copy link
Contributor

@lhanusov lhanusov left a 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 👍

@stianst stianst force-pushed the testsuite-poc-service-account branch from 652a4b1 to 4c29af0 Compare July 22, 2024 13:11
@vmuzikar
Copy link
Contributor

@stianst Was this PR superseded?

@stianst stianst restored the testsuite-poc-service-account branch July 23, 2024 11:12
@stianst stianst reopened this Jul 23, 2024
@stianst
Copy link
Contributor Author

stianst commented Jul 23, 2024

@stianst Was this PR superseded?

Nah, I deleted the branch by mistake. Re-opened now

@stianst stianst force-pushed the testsuite-poc-service-account branch from 4c29af0 to e7ab088 Compare July 23, 2024 12:02

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.
Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

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

@stianst stianst merged commit b4368b7 into keycloak:main Jul 24, 2024
@stianst stianst deleted the testsuite-poc-service-account branch July 24, 2024 11:14
stianst added a commit to stianst/keycloak that referenced this pull request Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testsuite PoC - Use service account when connecting to Admin APIs instead of admin user

5 participants