Problem/Motivation

Invalid unique identifier (authname) from invalid attribute name or missing attribute value throws an error.

Steps to reproduce

Proposed resolution

Add try/catch for graceful error handling and display a message to users.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3562330-simplesamlphp_auth-catch-empty-authname-error.patch860 bytesdrpldrp

Issue fork simplesamlphp_auth-3562330

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:

About issue forks

Comments

drpldrp created an issue. See original summary.

drpldrp’s picture

drpldrp commented
StatusFileSize
new 3562330-simplesamlphp_auth-catch-empty-authname-error.patch860 bytes
drpldrp’s picture

drpldrp commented
Category: Bug report » Feature request
garvitasakhrani’s picture

garvitasakhrani commented
Assigned: Unassigned » garvitasakhrani

I am working on it.

garvitasakhrani opened merge request !46

garvitasakhrani’s picture

garvitasakhrani commented
Assigned: garvitasakhrani » Unassigned
Status: Active » Needs review

I have added the try/catch around getAuthname() to handle missing or invalid SAML unique identifiers. The changes are pushed to the issue fork on branch 3562330-catch-invalid-unique.
Please let me know if any updates are required.