etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SAML2AuthManagerImpl: let the component return true on start 

- Return super.true() even if plugin is not enabled
- Return empty list when getCommands is called

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
This commit is contained in:
Rohit Yadav 2014-08-30 14:32:54 +02:00
parent 2bff5956a9
commit 784288eaab
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
plugins/user-authenticators/saml2/src/org/apache/cloudstack/saml/SAML2AuthManagerImpl.java
View File

@ -71,7 +71,10 @@ public class SAML2AuthManagerImpl extends AdapterBase implements SAML2AuthManage
@Override
public boolean start() {
return isSAMLPluginEnabled() && setup();
if (isSAMLPluginEnabled()) {
setup();
}
return super.start();
}
private boolean setup() {
@ -147,10 +150,10 @@ public class SAML2AuthManagerImpl extends AdapterBase implements SAML2AuthManage
@Override
public List<Class<?>> getAuthCommands() {
if (!isSAMLPluginEnabled()) {
return null;
}
List<Class<?>> cmdList = new ArrayList<Class<?>>();
if (!isSAMLPluginEnabled()) {
return cmdList;
}
cmdList.add(SAML2LoginAPIAuthenticatorCmd.class);
cmdList.add(SAML2LogoutAPIAuthenticatorCmd.class);
cmdList.add(GetServiceProviderMetaDataCmd.class);