Skip to content

Conversation

@TheForgottened
Copy link

The testcontainers library is giving support for a deprecated image of Keycloak.

This PR aims to give support to the new image of Keycloak, making sure it works independently of the tag used.

This has proved to be more difficult than it should've because of many reasons:

  1. The lack of proper docs from Keycloak
  2. Lots of differences between legacy and non-legacy Keycloak instances (with no documentation to explain better)
  3. API differences between versions / architectures

The code works 100% fine according to my tests and the tests added. I haven't tested with the legacy image tbh, but I have a hunch it might just work, except for version 17 and up.

Copy link
Contributor

@timbmg timbmg left a comment

Choose a reason for hiding this comment

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

LGTM

@TheForgottened
Copy link
Author

@timbmg anything missing on the PR that's stopping it from being merged?

@maltehedderich
Copy link
Contributor

@timbmg do you have the power to rerun the actions? I actually would like to use these changes, so would be nice if we could get them merged.

@eddumelendez
Copy link
Member

Thanks for your contribution, @TheForgottened . Currently, we are not able to maintain the module but there is an Module Catalog, which also list community modules from this repository. I think it would be great if the repository can be hosted under https://github.com/keycloak.

/cc @stianst

@disrupted
Copy link

@TheForgottened great contribution! would you be able to submit this to the Keycloak org as proposed by @eddumelendez?

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.

5 participants