Currently when you enable the simpleSAMLphp Authentication module, Federated ID is enabled immediately before any configuration is done. This can cause problems as Federated ID is enabled but not really is not working.
Suggestion: By default, Federated ID should be disabled in a new installation when the module is enabled.
There should be a checkbox to enable Federated ID in the configuration page. This should only be done when the configuration has been done. This checkbox can then be used to turn on and off Federated ID during the setup process
When the module is enable - there should be a message telling the user that federated id is currently disable and that you need to configure and enable it in the configuration page

Comments

geekwisdom’s picture

geekwisdom commented
Assigned: Unassigned » geekwisdom
Status: Active » Needs work

This makes sense. It could also help some folks who have been experiencing issues with being logged out as soon as the module is enabled.

geekwisdom’s picture

geekwisdom commented
Status: Needs work » Fixed

I have added an activation checkbox to the configuration page. Between this and the fix for issue #1302610 this issue should be resolved. This will be rolled into 7.x-1.1

geekwisdom’s picture

geekwisdom commented
Status: Fixed » Closed (fixed)

The next release of the 7.x module will correct this issue.