Attached is a patch impolementing the proposed resolution
Change status to needs review
Hiding patch file
This error seems to be related to this patch where a ! condition was changed to a condition without ! when the conditioned was refactored.
Attached is a patch re-adding the missing !
If a maintainer have the chance to look at this I will try to be quick in my response time to move this patch forward towards acceptance.
Thank you for the feedback.
I have provided a updatedb hook for the schema change and fixed the issue with settings.keycloak_i18n_mapping that should be settings.keycloak_i18n.mapping.
I left the initial config as it was since I belive that to be a reasonable default value.
The issue https://www.drupal.org/project/keycloak/issues/3390391 📌 Add keycloak support for multple instances from openid_connect 3.x Needs review implements support for multiple keycloak configurations and adds a global setting to choose which of the keycloak configurations that should be used to override the drupal login form.
Would it be possible for one of the maintainers to review this issue and see if this can be accepted?
The issue 3390391 (Add keycloak support for multple instances from openid_connect 3.x) 📌 Add keycloak support for multple instances from openid_connect 3.x Needs review aims to integrate this keycloak module in a way that support multiple clients. In issue 3390391 the functionality for single sign in ( 3382665 🐛 Replace Drupal login with Keycloak single sign-on (SSO) is not working Active ) and single sign out ( 3365872 ✨ [Openid_connect 3.x] - Support session check Active , 3107127 🐛 Can't get SingleSignOut to work Active ) is handled as well.
Is that approach something that could be considered a good path to reaching a stable keycoak module?
The functionality mentioned in the proposed resolution is implemented in the linked issue fork
Added google_tag-config cachetag to expected tags in tests.
Re-rolled patch against latest 2.0.x
Attaching path implementing suggested changes
Attaching patch that implements proposed solution
auth → created an issue. See original summary → .