-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add keycloak admin client #8315
Add keycloak admin client #8315
Conversation
Hi Stuart @stuartwdouglas, nice and simple, thanks. |
@gsmet Hi, I've removed a backport label for now as I'm not 100% sure it is a new feature or a bug fix :-) |
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.
Always a learning experience to see @stuartwdouglas code and how to do simple things that work in Quarkus.
@pedroigor I'm replacing this code now, pretty simple with the admin client :-) |
@sberyozkin Yeah, it should make things lot simpler when interacting to Keycloak REST API. |
@pedroigor I'm having issues in replacing the code across all the tests (this is only related to the use of the API, Stuart's PR is fine), so I will just update |
4fa0bd0
to
a7821db
Compare
Hi @pedroigor So I've updated |
@BuildStep | ||
ReflectiveHierarchyIgnoreWarningBuildItem marker(BuildProducer<AdditionalApplicationArchiveMarkerBuildItem> prod) { | ||
prod.produce(new AdditionalApplicationArchiveMarkerBuildItem("org/keycloak/admin/client/")); | ||
prod.produce(new AdditionalApplicationArchiveMarkerBuildItem("org/keycloak/representations")); |
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.
@stuartwdouglas I wonder if we should index the jar instead so that things are properly registered for reflection? Otherwise, there's a good chance things won't work very well in native.
Fixes #6350