libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

KEYCLOAK-7810 Fix NPE in Elytron SAML adapter

Browse source
This commit is contained in:
Hynek Mlnarik 2018-10-02 10:16:45 +02:00 committed by Hynek Mlnařík
parent c41bcddd8d
commit 211774ccbc
2 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
adapters/saml/wildfly-elytron/src/main/java/org/keycloak/adapters/saml/elytron/KeycloakHttpServerAuthenticationMechanism.java
View file

@ -79,7 +79,7 @@ class KeycloakHttpServerAuthenticationMechanism implements HttpServerAuthenticat
return; return;
} }
if (httpFacade.getRequest().getRelativePath().contains(deployment.getLogoutPage())) { if (deployment.getLogoutPage() != null && httpFacade.getRequest().getRelativePath().contains(deployment.getLogoutPage())) {
LOGGER.debugf("Ignoring request for [%s] and logout page [%s].", request.getRequestURI(), deployment.getLogoutPage()); LOGGER.debugf("Ignoring request for [%s] and logout page [%s].", request.getRequestURI(), deployment.getLogoutPage());
httpFacade.authenticationCompleteAnonymous(); httpFacade.authenticationCompleteAnonymous();
return; return;

1
testsuite/integration-arquillian/tests/base/src/test/resources/adapter-test/keycloak-saml/employee-acs/WEB-INF/keycloak-saml.xml
View file

@ -21,7 +21,6 @@
<SP entityID="http://localhost:8280/employee-acs/" <SP entityID="http://localhost:8280/employee-acs/"
sslPolicy="EXTERNAL" sslPolicy="EXTERNAL"
nameIDPolicyFormat="urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified" nameIDPolicyFormat="urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified"
logoutPage="/logout.jsp"
forceAuthentication="false"> forceAuthentication="false">
<PrincipalNameMapping policy="FROM_NAME_ID"/> <PrincipalNameMapping policy="FROM_NAME_ID"/>
<RoleIdentifiers> <RoleIdentifiers>