If you configure Drupal registration to allow anonymous registration, this module will automatically generate an account. If you require admin approval, it will create the account but then block it. If you disable registration completely and login with a user that doesn't exist yet you get two error messages and lots of PHP notices about undefined variables. This prevents the double error message and prevents the warning messages.

Warnings and errors:

warnings and errors screenshot

CommentFileSizeAuthor
#5 2728603-saml_sp-errors_on_login_without_registration_enabled-5.patch1.71 KBtizzo
#4 2728603-saml_sp-errors_on_login_without_registration_enabled-4.patch.patch3.83 KBtizzo
#3 2728603-saml_sp-errors_on_login_without_registration_enabled-3.patch3.45 KBtizzo
#2 2728603-saml_sp-errors_on_login_without_registration_enabled-2.patch1.72 KBtizzo
saml_sp_errors.jpg305.57 KBtizzo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tizzo created an issue. See original summary.

tizzo’s picture

tizzo CreditAttribution: tizzo commented
Status: Active » Needs review
FileSize
2728603-saml_sp-errors_on_login_without_registration_enabled-2.patch1.72 KB
tizzo’s picture

tizzo CreditAttribution: tizzo commented
FileSize
2728603-saml_sp-errors_on_login_without_registration_enabled-3.patch3.45 KB

Updating patch. The last one wouldn't apply to master.

tizzo’s picture

tizzo CreditAttribution: tizzo commented
FileSize
2728603-saml_sp-errors_on_login_without_registration_enabled-4.patch.patch3.83 KB

Big cleanup of that last patch which was super sloppy, sorry about that. This one should be good.

tizzo’s picture

tizzo CreditAttribution: tizzo commented
FileSize
2728603-saml_sp-errors_on_login_without_registration_enabled-5.patch1.71 KB

Inadvertently included the removal of a class that needs to stay. My bad. Nothing to see here folks…