From b8b4e25d5f1197f3a5af8cbcc1fee291fd3b7af2 Mon Sep 17 00:00:00 2001 From: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Date: Sat, 19 Dec 2020 08:55:25 +0000 Subject: [PATCH] Update to Keycloak 12.0.1 --- src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java | 156 ++------------------------------------------------- 1 files changed, 7 insertions(+), 149 deletions(-) diff --git a/src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java b/src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java index ed9d5fa..e83af7c 100644 --- a/src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java +++ b/src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java @@ -1,59 +1,30 @@ package org.keycloak.protocol.cas.endpoints; -import org.jboss.logging.Logger; import org.jboss.resteasy.annotations.cache.NoCache; -import org.jboss.resteasy.spi.HttpRequest; -import org.keycloak.common.ClientConnection; -import org.keycloak.events.Details; -import org.keycloak.events.Errors; import org.keycloak.events.EventBuilder; import org.keycloak.events.EventType; -import org.keycloak.models.*; +import org.keycloak.models.RealmModel; import org.keycloak.protocol.cas.CASLoginProtocol; -import org.keycloak.protocol.cas.representations.CASErrorCode; import org.keycloak.protocol.cas.utils.CASValidationException; -import org.keycloak.protocol.oidc.utils.RedirectUtils; -import org.keycloak.services.managers.AuthenticationManager; -import org.keycloak.services.managers.ClientSessionCode; import javax.ws.rs.GET; -import javax.ws.rs.core.*; +import javax.ws.rs.core.MediaType; +import javax.ws.rs.core.MultivaluedMap; +import javax.ws.rs.core.Response; -public class ValidateEndpoint { - protected static final Logger logger = Logger.getLogger(ValidateEndpoint.class); +public class ValidateEndpoint extends AbstractValidateEndpoint { private static final String RESPONSE_OK = "yes\n"; private static final String RESPONSE_FAILED = "no\n"; - @Context - protected KeycloakSession session; - - @Context - protected ClientConnection clientConnection; - - @Context - protected HttpRequest request; - - @Context - protected HttpHeaders headers; - - @Context - protected UriInfo uriInfo; - - protected RealmModel realm; - protected EventBuilder event; - protected ClientModel client; - protected AuthenticatedClientSessionModel clientSession; - public ValidateEndpoint(RealmModel realm, EventBuilder event) { - this.realm = realm; - this.event = event; + super(realm, event); } @GET @NoCache public Response build() { - MultivaluedMap<String, String> params = uriInfo.getQueryParameters(); + MultivaluedMap<String, String> params = session.getContext().getUri().getQueryParameters(); String service = params.getFirst(CASLoginProtocol.SERVICE_PARAM); String ticket = params.getFirst(CASLoginProtocol.TICKET_PARAM); boolean renew = params.containsKey(CASLoginProtocol.RENEW_PARAM); @@ -82,117 +53,4 @@ return Response.status(e.getStatus()).entity(RESPONSE_FAILED).type(MediaType.TEXT_PLAIN).build(); } - private void checkSsl() { - if (!uriInfo.getBaseUri().getScheme().equals("https") && realm.getSslRequired().isRequired(clientConnection)) { - throw new CASValidationException(CASErrorCode.INVALID_REQUEST, "HTTPS required", Response.Status.FORBIDDEN); - } - } - - private void checkRealm() { - if (!realm.isEnabled()) { - throw new CASValidationException(CASErrorCode.INTERNAL_ERROR, "Realm not enabled", Response.Status.FORBIDDEN); - } - } - - private void checkClient(String service) { - if (service == null) { - event.error(Errors.INVALID_REQUEST); - throw new CASValidationException(CASErrorCode.INVALID_REQUEST, "Missing parameter: " + CASLoginProtocol.SERVICE_PARAM, Response.Status.BAD_REQUEST); - } - - client = realm.getClients().stream() - .filter(c -> CASLoginProtocol.LOGIN_PROTOCOL.equals(c.getProtocol())) - .filter(c -> RedirectUtils.verifyRedirectUri(uriInfo, service, realm, c) != null) - .findFirst().orElse(null); - if (client == null) { - event.error(Errors.CLIENT_NOT_FOUND); - throw new CASValidationException(CASErrorCode.INVALID_SERVICE, "Client not found", Response.Status.BAD_REQUEST); - } - - if (!client.isEnabled()) { - event.error(Errors.CLIENT_DISABLED); - throw new CASValidationException(CASErrorCode.INVALID_SERVICE, "Client disabled", Response.Status.BAD_REQUEST); - } - - event.client(client.getClientId()); - - session.getContext().setClient(client); - } - - private void checkTicket(String ticket, boolean requireReauth) { - if (ticket == null) { - event.error(Errors.INVALID_CODE); - throw new CASValidationException(CASErrorCode.INVALID_REQUEST, "Missing parameter: " + CASLoginProtocol.TICKET_PARAM, Response.Status.BAD_REQUEST); - } - if (!ticket.startsWith(CASLoginProtocol.SERVICE_TICKET_PREFIX)) { - event.error(Errors.INVALID_CODE); - throw new CASValidationException(CASErrorCode.INVALID_TICKET_SPEC, "Malformed service ticket", Response.Status.BAD_REQUEST); - } - - String code = ticket.substring(CASLoginProtocol.SERVICE_TICKET_PREFIX.length()); - - String[] parts = code.split("\\."); - if (parts.length == 4) { - event.detail(Details.CODE_ID, parts[2]); - } - - ClientSessionCode.ParseResult<AuthenticatedClientSessionModel> parseResult = ClientSessionCode.parseResult(code, session, realm, AuthenticatedClientSessionModel.class); - if (parseResult.isAuthSessionNotFound() || parseResult.isIllegalHash()) { - event.error(Errors.INVALID_CODE); - - // Attempt to use same code twice should invalidate existing clientSession - AuthenticatedClientSessionModel clientSession = parseResult.getClientSession(); - if (clientSession != null) { - clientSession.setUserSession(null); - } - - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "Code not valid", Response.Status.BAD_REQUEST); - } - - clientSession = parseResult.getClientSession(); - - if (!parseResult.getCode().isValid(AuthenticatedClientSessionModel.Action.CODE_TO_TOKEN.name(), ClientSessionCode.ActionType.CLIENT)) { - event.error(Errors.INVALID_CODE); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "Code is expired", Response.Status.BAD_REQUEST); - } - - clientSession.setNote(CASLoginProtocol.SESSION_SERVICE_TICKET, ticket); - parseResult.getCode().setAction(null); - - if (requireReauth && AuthenticationManager.isSSOAuthentication(clientSession)) { - event.error(Errors.SESSION_EXPIRED); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "Interactive authentication was requested but not performed", Response.Status.BAD_REQUEST); - } - - UserSessionModel userSession = clientSession.getUserSession(); - - if (userSession == null) { - event.error(Errors.USER_SESSION_NOT_FOUND); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "User session not found", Response.Status.BAD_REQUEST); - } - - UserModel user = userSession.getUser(); - if (user == null) { - event.error(Errors.USER_NOT_FOUND); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "User not found", Response.Status.BAD_REQUEST); - } - if (!user.isEnabled()) { - event.error(Errors.USER_DISABLED); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "User disabled", Response.Status.BAD_REQUEST); - } - - event.user(userSession.getUser()); - event.session(userSession.getId()); - - if (!client.getClientId().equals(clientSession.getClient().getClientId())) { - event.error(Errors.INVALID_CODE); - throw new CASValidationException(CASErrorCode.INVALID_SERVICE, "Auth error", Response.Status.BAD_REQUEST); - } - - if (!AuthenticationManager.isSessionValid(realm, userSession)) { - event.error(Errors.USER_SESSION_NOT_FOUND); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "Session not active", Response.Status.BAD_REQUEST); - } - - } } -- Gitblit v1.9.1