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

Update docs regarding OpenID Connect Logout #749

Closed
wants to merge 1 commit into from

Conversation

jjojo
Copy link

@jjojo jjojo commented Oct 10, 2022

Keycloak v18+ implements OpenID Connect Logout (https://www.keycloak.org/2022/04/keycloak-1800-released) and it requires post_logout_redirect_uri to be set to properly sign out. This terraform provider can handle this by utilizing the "extra_config" option.

Keycloak v18+ implements OpenID Connect Logout (https://www.keycloak.org/2022/04/keycloak-1800-released) and it requires post_logout_redirect_uri to be set to properly sign out. This terraform provider can handle this by utilizing the "extra_config" option.
@earnubs
Copy link

earnubs commented Nov 29, 2022

Using extra_config for this appears to fail during an upgrade:

ERROR: Migration failed for change set META-INF/jpa-changelog-19.0.0.xml::19.0.0-10135::keycloak:
  Reason: liquibase.exception.DatabaseException: ERROR: duplicate key value violates unique constraint \"constraint_3c\"
  Detail: Key (client_id, name)=(18e9c7c6-2224-4a05-9a90-417ff499daca, post.logout.redirect.uris) already exists. [Failed SQL: (0) INSERT INTO public.client_attributes (CLIENT_ID, NAME, VALUE) VALUES ('18e9c7c6-2224-4a05-9a90-417ff499daca', 'post.logout.redirect.uris', '+')]

@mrparkers
Copy link
Contributor

Thanks for opening this PR, but I decided to add top-level support for this attribute instead in #777.

@mrparkers mrparkers closed this Dec 4, 2022
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.

3 participants