Skip to content
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

[DRAFT] Feature: Support RealmEvents and RealmUserProfile #153

Conversation

b509
Copy link
Contributor

@b509 b509 commented Aug 23, 2024

Description of your changes

Add support for keycloak_realm_events and keycloak_realm_user_profile

I have:

  • Read and followed Crossplane's [contribution process].
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Tested it on my local k3d instance with default keycloak (https://github.com/codecentric/helm-charts/tree/master/charts/keycloakx).

Open question / Why is this a draft

During testing I discovered that sometime when trying to update the UserProfile in a way that throws a 400 Bad Request error in the API, the resource is nevertheless reported as synced and ready (which was not successful). I would be grateful if someone else could have a look and see if they have the same experience.

How to reproduce: Take the userprofile from examples/realmuserprofile.yaml and remove the "username" and/or "email" attribute. It should give an error from Keycloak stating

400 Bad Request. Response body: {"errorMessage":"[The attribute 'username' can not be removed, The attribute 'email' can not be removed]"}

@b509
Copy link
Contributor Author

b509 commented Aug 28, 2024

Replaced by #154 with correct signed-of annotation :)

@b509 b509 closed this Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants