Problem/Motivation

LDAP 4.0-beta-2 has a small change that causes LDAP SSO Auth to no longer work.

class: \Drupal\ldap_authentication\Controller\LoginValidatorSso

LoginValidatorSso::processLogin() no longer accepts a username; the method takes no arguments.

LoginValidatorSso::setAuthname() has been added.

Steps to reproduce

Install LDAP 4.0-beta-2 and LDAP SSO Auth 2.1

Proposed resolution

Change LdapSsoAuthAuthentication.php Line 169

- $this->validator->processLogin(Html::escape($remote_user));
+ $this->validator->setAuthname(Html::escape($remote_user));
+ $this->validator->processLogin();

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 3209119-3.patch1.01 KBbluegeek9
#2 3209119-2.patch1.06 KBbluegeek9
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

bluegeek9 created an issue. See original summary.

bluegeek9’s picture

StatusFileSize
new1.06 KB

This patch is meant for dev and handles the change in the LDAP module and increases the minimum version to 4.0.0-beta2.

bluegeek9’s picture

StatusFileSize
new1.01 KB

This patch is for version 2.1, handles the change in the LDAP module, and increases the minimum version to 4.0.0-beta2.

  • bluegeek9 committed b9c184ac on 8.x-2.x
    Issue #3209119 by bluegeek9: LDAP 4.0-beta-2 Compatibility
    

  • bluegeek9 authored e81516f1 on 8.x-2.x
    Issue #3209119 by bluegeek9: LDAP 4.0-beta-2 Compatibility
    
bluegeek9’s picture

Status: Active » Fixed
dj1999’s picture

Status: Fixed » Closed (fixed)