From 461e1c1f749d8eaf02015073abff1bb8a20912bb Mon Sep 17 00:00:00 2001 From: Jacek Kowalski <Jacek@jacekk.info> Date: Wed, 23 Mar 2022 08:58:04 +0000 Subject: [PATCH] Update github.pulls.create usage in update-deps workflow --- src/main/java/org/keycloak/protocol/cas/endpoints/ValidateEndpoint.java | 178 ++--------------------------------------------------------- 1 files changed, 7 insertions(+), 171 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 c250578..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,60 +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 java.lang.reflect.Method; +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); @@ -83,138 +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; - try { - // Keycloak >3.4 branch: Parameter event was added to ClientSessionCode.parseResult - Method parseResultMethod = ClientSessionCode.class.getMethod("parseResult", - String.class, KeycloakSession.class, RealmModel.class, EventBuilder.class, Class.class); - parseResult = (ClientSessionCode.ParseResult<AuthenticatedClientSessionModel>) parseResultMethod.invoke( - null, code, session, realm, event, AuthenticatedClientSessionModel.class); - } catch (ReflectiveOperationException e) { - // Keycloak <=3.3 branch - 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(); - - try { - // Keycloak >3.4 branch: Method isExpiredToken was added - Method isExpiredToken = ClientSessionCode.ParseResult.class.getMethod("isExpiredToken"); - if ((Boolean) isExpiredToken.invoke(parseResult)) { - event.error(Errors.EXPIRED_CODE); - throw new CASValidationException(CASErrorCode.INVALID_TICKET, "Code is expired", Response.Status.BAD_REQUEST); - } - } catch (ReflectiveOperationException e) { - // Keycloak <=3.3 branch - 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); - } - - parseResult.getCode().setAction(null); - } - - clientSession.setNote(CASLoginProtocol.SESSION_SERVICE_TICKET, ticket); - - 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