Michal Vavřík
Michal Vavřík
hey @pedroigor please review if you would like, otherwise in terms of what is changed, changes are not specific to Keycloak (it's a CDI feature...), so @sberyozkin is right reviewer...
@sberyozkin I pushed it by accident as I need to fix some tests, docs and write more tests, but as it's already there, you can look at the builder and...
Done, the `DynamicTenantConfigPolicyEnforcerTest` tests new builder comprehensively.
> Hi @michalvavrik, can you please type a builder sequence showing all the supported properties being set Please have a look at the `io.quarkus.it.keycloak.DynamicTenantConfigPolicyEnforcerTest#assertEveryConfigPropertyCanBeSet` added in this PR. You can...
ping @sberyozkin
I've rebased on the current main and resolved merge conflicts just in case @sberyozkin find a time to review.
@sberyozkin please have a look; I won't resolve merge conflicts again as I'm bit worried it's wasting of CI resources. Will do it when you find a time for this
Merge conflicts were about TLS registry, the rest of this PR still applies. Resolved.
I am going to have a look. I cannot tell whether https://github.com/quarkusio/quarkus/compare/main...ryandens:quarkus:ryandens/41223/oidc-client-secret is a good solution without trying to figure it out myself. Sure, it might end-up same + tests....
Some changes here (Moved required options to parameters: SECRET, NAME, VALUE) are not backwards compatible, but I can't find it documented in https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.13 and this PR does not have a...