Check logic in function _simplesamlphp_auth_isEnabled()

backdrop-contrib / simplesamlphp_auth

Support SAML for authentication of users. The module will auto-provision user accounts and dynamically assign roles in Backdrop CMS if you want it to.

https://backdropcms.org/project/simplesamlphp_auth

GNU General Public License v2.0

2 stars 3 forks source link

Check logic in function _simplesamlphp_auth_isEnabled() #15

Closed laryn closed 1 year ago

laryn commented 1 year ago

On a local while running some tests I am seeing this:

Error: Call to a member function isAuthenticated() on null in simplesamlphp_auth_user_insert() (line 374 of /app/web/modules/contrib/simplesamlphp_auth/simplesamlphp_auth.module).

I have the module enabled but the "activated" checkbox unchecked. I think maybe the logic in _simplesamlphp_auth_isEnabled() needs some slight tweaking. I'll test something and make a PR for your consideration.