Problem/Motivation

Some configurations change have no effect, like enabling multi-language support.

2 main reason for that, hardcoded config name 'openid_connect.settings.keycloak' in KeycloakService.php and KeycloakController.php

also the check for the setting being enabled wich should check for 'status' instead of 'enabled'

Steps to reproduce

Install the module, try to set up multi-language support, see kc_local not being set in the keycloak login url

Proposed resolution

The patch attached fixed it for my project, it will work only with a openID Connect client named "keycloak" , this is because now we can have multiple keycloak configuration instances, as seen in this issue https://www.drupal.org/project/keycloak/issues/3390391 (that should be fixes first to have a clean definitive patch)

Remaining tasks

- fix above issue
- remake a patch that fix remaining problems

CommentFileSizeAuthor
keycloak-config-fix.patch2.04 KBthony

Comments

thony created an issue. See original summary.

thony’s picture

Issue summary: View changes
joseph.olstad’s picture

The above patch no longer applies to 2.2.x dev.

See:
#3390391: Add keycloak support for multple instances from openid_connect 3.x

The changes in 3390391 are included in 2.2.3-rc1

Please test 2.2.3-rc1 to see if 3497660 is still needed.

joseph.olstad’s picture

Status: Active » Needs work
joseph.olstad’s picture

Status: Needs work » Needs review
joseph.olstad’s picture

Status: Needs review » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.