Details
diff --git a/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java b/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
index fba3aa9..96e922d 100755
--- a/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
+++ b/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
@@ -8,7 +8,7 @@ import org.keycloak.models.picketlink.PicketlinkKeycloakSessionFactory;
import org.keycloak.models.picketlink.mappings.ApplicationEntity;
import org.keycloak.models.picketlink.mappings.RealmEntity;
import org.keycloak.services.utils.PropertiesManager;
-import org.keycloak.social.SocialRequestManager;
+import org.keycloak.services.managers.SocialRequestManager;
import org.picketlink.idm.PartitionManager;
import org.picketlink.idm.config.IdentityConfigurationBuilder;
import org.picketlink.idm.internal.DefaultPartitionManager;
diff --git a/services/src/main/java/org/keycloak/services/resources/SocialResource.java b/services/src/main/java/org/keycloak/services/resources/SocialResource.java
index 4d337b5..85528c2 100755
--- a/services/src/main/java/org/keycloak/services/resources/SocialResource.java
+++ b/services/src/main/java/org/keycloak/services/resources/SocialResource.java
@@ -66,7 +66,7 @@ import org.keycloak.social.SocialConstants;
import org.keycloak.social.SocialProvider;
import org.keycloak.social.SocialProviderConfig;
import org.keycloak.social.SocialProviderException;
-import org.keycloak.social.SocialRequestManager;
+import org.keycloak.services.managers.SocialRequestManager;
import org.keycloak.social.SocialUser;
/**
@@ -186,7 +186,6 @@ public class SocialResource {
MultivaluedMap<String, String> formData = fillRegistrationFormWithSocialData(socialUser);
RequestDetailsBuilder reqDetailsBuilder = RequestDetailsBuilder.createFromRequestDetails(requestData);
- reqDetailsBuilder.putSocialAttribute(SocialConstants.ATTR_SOCIAL_LINK, socialLink);
String requestId = UUID.randomUUID().toString();
socialRequestManager.addRequest(requestId, reqDetailsBuilder.build());
@@ -281,7 +280,6 @@ public class SocialResource {
String scope = requestData.getClientAttribute("scope");
String state = requestData.getClientAttribute("state");
String redirectUri = requestData.getClientAttribute("redirectUri");
- SocialLinkModel socialLink = (SocialLinkModel)requestData.getSocialAttribute(SocialConstants.ATTR_SOCIAL_LINK);
Response response1 = tokenService.processRegisterImpl(clientId, scope, state, redirectUri, formData, true);
@@ -297,7 +295,7 @@ public class SocialResource {
// Normally shouldn't happen
throw new IllegalStateException("User " + username + " not found in the realm");
}
- realm.addSocialLink(user, socialLink);
+ realm.addSocialLink(user, new SocialLinkModel(requestData.getProviderId(), username));
// Expire cookie and invalidate requestData
String cookiePath = Urls.socialBase(uriInfo.getBaseUri()).build().getPath();
diff --git a/social/core/src/main/java/org/keycloak/social/AuthCallback.java b/social/core/src/main/java/org/keycloak/social/AuthCallback.java
index 8e21a21..d9c579a 100644
--- a/social/core/src/main/java/org/keycloak/social/AuthCallback.java
+++ b/social/core/src/main/java/org/keycloak/social/AuthCallback.java
@@ -28,11 +28,11 @@ import java.util.Map;
*/
public class AuthCallback {
- private Map<String, Object> attributes;
+ private Map<String, String> attributes;
private Map<String, String[]> queryParams;
- public AuthCallback(Map<String, Object> attributes, Map<String, String[]> queryParams) {
+ public AuthCallback(Map<String, String> attributes, Map<String, String[]> queryParams) {
this.attributes = attributes;
this.queryParams = queryParams;
}
diff --git a/social/core/src/main/java/org/keycloak/social/AuthRequest.java b/social/core/src/main/java/org/keycloak/social/AuthRequest.java
index 69731e9..a88d805 100644
--- a/social/core/src/main/java/org/keycloak/social/AuthRequest.java
+++ b/social/core/src/main/java/org/keycloak/social/AuthRequest.java
@@ -33,9 +33,9 @@ public class AuthRequest {
private URI authUri;
- private Map<String, Object> attributes;
+ private Map<String, String> attributes;
- AuthRequest(String id, URI authUri, Map<String, Object> attributes) {
+ AuthRequest(String id, URI authUri, Map<String, String> attributes) {
this.id = id;
this.authUri = authUri;
this.attributes = attributes;
@@ -49,7 +49,7 @@ public class AuthRequest {
return authUri;
}
- public Map<String, Object> getAttributes() {
+ public Map<String, String> getAttributes() {
return attributes;
}
diff --git a/social/core/src/main/java/org/keycloak/social/AuthRequestBuilder.java b/social/core/src/main/java/org/keycloak/social/AuthRequestBuilder.java
index c5dd66a..600783d 100644
--- a/social/core/src/main/java/org/keycloak/social/AuthRequestBuilder.java
+++ b/social/core/src/main/java/org/keycloak/social/AuthRequestBuilder.java
@@ -33,7 +33,7 @@ public class AuthRequestBuilder {
private UriBuilder b;
- private Map<String, Object> attributes;
+ private Map<String, String> attributes;
private String id;
@@ -44,7 +44,7 @@ public class AuthRequestBuilder {
AuthRequestBuilder req = new AuthRequestBuilder();
req.id = id;
req.b = UriBuilder.fromUri(path);
- req.attributes = new HashMap<String, Object>();
+ req.attributes = new HashMap<String, String>();
return req;
}
@@ -53,7 +53,7 @@ public class AuthRequestBuilder {
return this;
}
- public AuthRequestBuilder setAttribute(String name, Object value) {
+ public AuthRequestBuilder setAttribute(String name, String value) {
attributes.put(name, value);
return this;
}
diff --git a/social/core/src/main/java/org/keycloak/social/RequestDetails.java b/social/core/src/main/java/org/keycloak/social/RequestDetails.java
index f77f59b..0476a64 100644
--- a/social/core/src/main/java/org/keycloak/social/RequestDetails.java
+++ b/social/core/src/main/java/org/keycloak/social/RequestDetails.java
@@ -32,9 +32,9 @@ public class RequestDetails {
private Map<String, String> clientAttributes;
- private Map<String, Object> socialAttributes;
+ private Map<String, String> socialAttributes;
- RequestDetails(String providerId, Map<String, String> clientAttributes, Map<String, Object> socialAttributes) {
+ RequestDetails(String providerId, Map<String, String> clientAttributes, Map<String, String> socialAttributes) {
this.providerId = providerId;
this.clientAttributes = clientAttributes;
this.socialAttributes = socialAttributes;
@@ -52,11 +52,11 @@ public class RequestDetails {
return clientAttributes;
}
- public Object getSocialAttribute(String name) {
+ public String getSocialAttribute(String name) {
return socialAttributes.get(name);
}
- public Map<String, Object> getSocialAttributes() {
+ public Map<String, String> getSocialAttributes() {
return socialAttributes;
}
diff --git a/social/core/src/main/java/org/keycloak/social/RequestDetailsBuilder.java b/social/core/src/main/java/org/keycloak/social/RequestDetailsBuilder.java
index aa86fbe..0c2c73d 100644
--- a/social/core/src/main/java/org/keycloak/social/RequestDetailsBuilder.java
+++ b/social/core/src/main/java/org/keycloak/social/RequestDetailsBuilder.java
@@ -33,7 +33,7 @@ public class RequestDetailsBuilder {
private Map<String, String> clientAttributes;
- private Map<String, Object> socialAttributes;
+ private Map<String, String> socialAttributes;
private RequestDetailsBuilder() {
}
@@ -42,7 +42,7 @@ public class RequestDetailsBuilder {
RequestDetailsBuilder req = new RequestDetailsBuilder();
req.providerId = providerId;
req.clientAttributes = new HashMap<String, String>();
- req.socialAttributes = new HashMap<String, Object>();
+ req.socialAttributes = new HashMap<String, String>();
return req;
}
@@ -51,7 +51,7 @@ public class RequestDetailsBuilder {
req.providerId = from.getProviderId();
req.clientAttributes = new HashMap<String, String>();
req.clientAttributes.putAll(from.getClientAttributes());
- req.socialAttributes = new HashMap<String, Object>();
+ req.socialAttributes = new HashMap<String, String>();
req.socialAttributes.putAll(from.getSocialAttributes());
return req;
}
@@ -66,12 +66,12 @@ public class RequestDetailsBuilder {
return this;
}
- public RequestDetailsBuilder putSocialAttribute(String name, Object value) {
+ public RequestDetailsBuilder putSocialAttribute(String name, String value) {
socialAttributes.put(name, value);
return this;
}
- public RequestDetailsBuilder putSocialAttributes(Map<String, Object> attributes) {
+ public RequestDetailsBuilder putSocialAttributes(Map<String, String> attributes) {
socialAttributes.putAll(attributes);
return this;
}
diff --git a/social/core/src/main/java/org/keycloak/social/SocialProviderConfig.java b/social/core/src/main/java/org/keycloak/social/SocialProviderConfig.java
index d6d1276..4989861 100644
--- a/social/core/src/main/java/org/keycloak/social/SocialProviderConfig.java
+++ b/social/core/src/main/java/org/keycloak/social/SocialProviderConfig.java
@@ -50,8 +50,4 @@ public class SocialProviderConfig {
return secret;
}
- public void setSecret(String secret) {
- this.secret = secret;
- }
-
}