Added instructions and support for have script policy provider workin…#316
Added instructions and support for have script policy provider workin…#316mposolda merged 1 commit intokeycloak:mainfrom
Conversation
|
This is draft PR related to keycloak/keycloak#11322 . See that PR for more details. |
| System Requirements | ||
| ------------------- | ||
|
|
||
| You need to have Keycloak 18.0.Final running. |
There was a problem hiding this comment.
perhaps better "at least Keycloak 18.0.Final" or sth? I'd avoid strict version pinning in a general example.
There was a problem hiding this comment.
Yes, true. Thanks for these points. I've copy/pasted this from the README of some other example :-) Similarly the Java version.
It seems that README files of the quickstarts, which are deployable to Keycloak server, may need some revision as they are currently target just for the Keycloak on Wildfly.
This particular quickstart didn't have README file like the other quickstarts have. So I've added README file, but for this README, I've added lots of TODO and I focused only on parts specific to instructions for deployment on Keycloak on Wildfly VS Keycloak on Quarkus as this is the most important part for the particular use-case addressed here for the scripts providers. I can properly update the README file and remove all TODOs if we agree that my approach for script providers work.
a9655aa to
362fd59
Compare
5735385 to
b98eeaf
Compare
…g on Java 17 Closes keycloak#324
b98eeaf to
e267b11
Compare
|
@pedroigor @DGuhr Do you please have a chance to re-review this? Corresponding main codebase PR is also ready for the review: keycloak/keycloak#11322 . |
…g on Java 17