Handle case of only email before @ returned. And match email via user_load_by_mail, not custom SQL.

Created on 26 January 2023, almost 2 years ago

Updated 7 July 2023, over 1 year ago

Some SAML setups and Drupal 7 setups cannot pull the full email from the SAML provider and only pull the email name before the '@'.
To get the User / UID, the SQL provided in saml_sp_drupal_login.module does not work with all situations in Drupal 7.

Modify logic that gets nameID for the email case.
Just use the function user_load_by_mail instead of the SQL, that while should do the same thing, will not always in certain configurations. https://api.drupal.org/api/drupal/modules!user!user.module/function/user...

Patch will be provided.

✨ Feature request

Status

Downport

Version

3.0

Component

Code

Created by

🇺🇸United States darrell_ulm

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @darrell_ulm
Comment almost 2 years ago →
🇺🇸United States darrell_ulm
Patch provided here, works for me and fixes how email nameID works.
Comment almost 2 years ago →
🇺🇸United States darrell_ulm
Comment almost 2 years ago →
🇺🇸United States darrell_ulm
Status changed to Downport almost 2 years ago9:00pm 26 January 2023
Comment almost 2 years ago →
🇺🇸United States darrell_ulm
Comment over 1 year ago →
🇺🇸United States jrglasgow Idaho
is you want to create a form and a merge request I can look into merging it in.

Production build 0.71.5 2024