Merge pull request #1001 from pedroigor/master

[KEYCLOAK-883] - Reverting change that redirects to error page when fede...
keycloak · Feb 26, 2015 · cd39d52 · cd39d52
2 parents 69da51c + 17653e7
commit cd39d52
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/services/src/main/java/org/keycloak/services/resources/IdentityBrokerService.java b/services/src/main/java/org/keycloak/services/resources/IdentityBrokerService.java
@@ -323,7 +323,7 @@ private Response performLocalAuthentication(FederatedIdentity updatedIdentity, C
                     federatedUser.addRequiredAction(UPDATE_PROFILE);
                 }
             } catch (Exception e) {
-                return redirectToErrorPage(e.getMessage(), e);
+                return redirectToLoginPage(e, clientCode);
             }
         }
 
@@ -447,6 +447,20 @@ private Response redirectToErrorPage(String message, Throwable throwable) {
         return Flows.forwardToSecurityFailurePage(this.session, this.realmModel, this.uriInfo, message);
     }
 
+    private Response redirectToLoginPage(Throwable t, ClientSessionCode clientCode) {
+        String message = t.getMessage();
+
+        if (message == null) {
+            message = "Unexpected error when authenticating with identity provider";
+        }
+
+        fireErrorEvent(message);
+        return Flows.forms(this.session, this.realmModel, clientCode.getClientSession().getClient(), this.uriInfo)
+                .setClientSessionCode(clientCode.getCode())
+                .setError(message)
+                .createLogin();
+    }
+
     private Response badRequest(String message) {
         fireErrorEvent(message);
         return Flows.errors().error(message, Status.BAD_REQUEST);