For users with sufficient permissions, an option should be added to the user registration & profile fields for allowing/disallowing authentication through SimpleSAML.

CommentFileSizeAuthor
#1 simplesamlphp_auth-auth-option-2205901-1.patch2.24 KBewisuri

Comments

ewisuri’s picture

ewisuri commented
StatusFileSize
new simplesamlphp_auth-auth-option-2205901-1.patch2.24 KB

Adding patch for issue.

geekwisdom’s picture

geekwisdom commented
Status: Active » Needs review

Good idea.

grasmash’s picture

grasmash commented

I've created a 7.x-3.x branch of this module with significant refactoring. It includes patches from the following issues:

#2190487: User_login form should have a federated link
#2213283: When simplesamlphp_auth_activate is 0 watchdog messages flood the syslog/dblog
#2221679: Destruction of session causes fatal error
#2020009: Session cookie being set, breaking through Varnish cache.
#1824194: hook_user_insert issues when user adding a new user is logged in via sso
#2205901: Allow SimpleSAML authentication on a per-user basis

https://github.com/grasmash/simplesamlphp_auth/tree/7.x-3.x

Given that so much work has been done (and it will not cleanly merge with 7.x-2.x) I propose that a new 7.x-3.x-dev branch be added to the module.

  • snufkin committed 15065b5 on 7.x-3.x authored by grasmash 
    Issue #2205901 by madmatter23, Ability to enable SAML per user.
snufkin’s picture

snufkin commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Status: Needs review » Fixed

Merged Matts branch into 3.x, this issue is fixed there.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.