Merge changes

master
Jens Pelzetter 2019-11-30 09:23:23 +01:00
parent a55d61bd49
commit 392e6578ce
1 changed files with 17 additions and 1 deletions

View File

@ -55,6 +55,7 @@ public class SamlLoginModule implements LoginModule {
@Override
public boolean login() throws LoginException {
<<<<<<< HEAD
final HttpServletRequest httpRequest = getRequest();
final HttpServletResponse httpResponse = getResponse();
@ -64,6 +65,15 @@ public class SamlLoginModule implements LoginModule {
httpRequest,
httpResponse);
} catch (SettingsException ex) {
=======
final HttpServletRequest request = getRequest();
final HttpServletResponse response = getResponse();
final Auth auth;
try {
auth = new Auth(request, response);
} catch (IOException | SettingsException | Error ex) {
>>>>>>> SAML support for CCM
LOGGER.error("SAML Login failed.", ex);
throw new LoginException("SAML Login failed. Configuration error?");
}
@ -87,6 +97,9 @@ public class SamlLoginModule implements LoginModule {
@Override
public boolean commit() throws LoginException {
if (userId != null) {
subject.getPrincipals().add(new PartyPrincipal(userId));
}
return true;
}
@ -97,6 +110,10 @@ public class SamlLoginModule implements LoginModule {
@Override
public boolean logout() throws LoginException {
getRequest().getSession().invalidate();
return true;
}
protected HttpServletRequest getRequest() throws LoginException {
try {
@ -139,5 +156,4 @@ public class SamlLoginModule implements LoginModule {
);
}
}
}