From 755fd78fa0ee0f2a67417a119382c63e02c1091e Mon Sep 17 00:00:00 2001
From: Alexandre Rocha Wendling <alexandrerw@celepar.pr.gov.br>
Date: Tue, 16 Jul 2024 14:15:23 +0000
Subject: [PATCH] Proxy ticket service and proxy ticket validation Proxy endpoints improvements suggested by Jacek Kowalski Add ticket type to storage key Rename isreuse to isReusable Remove "parsing" of "codeUUID" that is String, not UUID Improve error reporting in CAS ticket validation
---
src/main/java/org/keycloak/protocol/cas/CASLoginProtocolFactory.java | 61 ++++++++++++------------------
1 files changed, 24 insertions(+), 37 deletions(-)
diff --git a/src/main/java/org/keycloak/protocol/cas/CASLoginProtocolFactory.java b/src/main/java/org/keycloak/protocol/cas/CASLoginProtocolFactory.java
index 57745b8..fb7a7b5 100644
--- a/src/main/java/org/keycloak/protocol/cas/CASLoginProtocolFactory.java
+++ b/src/main/java/org/keycloak/protocol/cas/CASLoginProtocolFactory.java
@@ -2,23 +2,21 @@
import org.jboss.logging.Logger;
import org.keycloak.events.EventBuilder;
-import org.keycloak.models.*;
+import org.keycloak.models.ClientModel;
+import org.keycloak.models.KeycloakSession;
+import org.keycloak.models.ProtocolMapperModel;
+import org.keycloak.models.RealmModel;
import org.keycloak.protocol.AbstractLoginProtocolFactory;
import org.keycloak.protocol.LoginProtocol;
-import org.keycloak.protocol.ProtocolMapperUtils;
import org.keycloak.protocol.cas.mappers.FullNameMapper;
import org.keycloak.protocol.cas.mappers.UserAttributeMapper;
import org.keycloak.protocol.cas.mappers.UserPropertyMapper;
import org.keycloak.representations.idm.ClientRepresentation;
-import org.keycloak.representations.idm.ClientTemplateRepresentation;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
-
-import static org.keycloak.protocol.oidc.mappers.OIDCAttributeMapperHelper.JSON_TYPE;
-import static org.keycloak.protocol.oidc.mappers.OIDCAttributeMapperHelper.TOKEN_CLAIM_NAME;
public class CASLoginProtocolFactory extends AbstractLoginProtocolFactory {
private static final Logger logger = Logger.getLogger(CASLoginProtocolFactory.class);
@@ -43,49 +41,43 @@
}
@Override
- public List<ProtocolMapperModel> getBuiltinMappers() {
+ public Map<String, ProtocolMapperModel> getBuiltinMappers() {
return builtins;
}
- @Override
- public List<ProtocolMapperModel> getDefaultBuiltinMappers() {
- return defaultBuiltins;
- }
-
- static List<ProtocolMapperModel> builtins = new ArrayList<>();
+ static Map<String, ProtocolMapperModel> builtins = new HashMap<>();
static List<ProtocolMapperModel> defaultBuiltins = new ArrayList<>();
static {
ProtocolMapperModel model;
- model = UserPropertyMapper.create(EMAIL, "email", "mail", "String",
- true, EMAIL_CONSENT_TEXT);
- builtins.add(model);
+ model = UserPropertyMapper.create(EMAIL, "email", "mail", "String");
+ builtins.put(EMAIL, model);
defaultBuiltins.add(model);
- model = UserPropertyMapper.create(GIVEN_NAME, "firstName", "givenName", "String",
- true, GIVEN_NAME_CONSENT_TEXT);
- builtins.add(model);
+ model = UserPropertyMapper.create(GIVEN_NAME, "firstName", "givenName", "String");
+ builtins.put(GIVEN_NAME, model);
defaultBuiltins.add(model);
- model = UserPropertyMapper.create(FAMILY_NAME, "lastName", "sn", "String",
- true, FAMILY_NAME_CONSENT_TEXT);
- builtins.add(model);
+ model = UserPropertyMapper.create(FAMILY_NAME, "lastName", "sn", "String");
+ builtins.put(FAMILY_NAME, model);
defaultBuiltins.add(model);
model = UserPropertyMapper.create(EMAIL_VERIFIED,
"emailVerified",
- "emailVerified", "boolean",
- false, EMAIL_VERIFIED_CONSENT_TEXT);
- builtins.add(model);
+ "emailVerified", "boolean");
+ builtins.put(EMAIL_VERIFIED, model);
model = UserAttributeMapper.create(LOCALE,
"locale",
"locale", "String",
- false, LOCALE_CONSENT_TEXT,
false);
- builtins.add(model);
+ builtins.put(LOCALE, model);
- model = FullNameMapper.create(FULL_NAME, "cn",
- true, FULL_NAME_CONSENT_TEXT);
- builtins.add(model);
+ model = FullNameMapper.create(FULL_NAME, "cn");
+ builtins.put(FULL_NAME, model);
defaultBuiltins.add(model);
+ }
+
+ @Override
+ protected void createDefaultClientScopesImpl(RealmModel newRealm) {
+ // no-op
}
@Override
@@ -94,8 +86,8 @@
}
@Override
- public Object createProtocolEndpoint(RealmModel realm, EventBuilder event) {
- return new CASLoginProtocolService(realm, event);
+ public Object createProtocolEndpoint(KeycloakSession session, EventBuilder event) {
+ return new CASLoginProtocolService(session, event);
}
@Override
@@ -115,10 +107,5 @@
if (rep.getAdminUrl() == null && rep.getRootUrl() != null) {
newClient.setManagementUrl(rep.getRootUrl());
}
- }
-
- @Override
- public void setupTemplateDefaults(ClientTemplateRepresentation clientRep, ClientTemplateModel newClient) {
-
}
}
--
Gitblit v1.9.1