Added error handling for register when password spec is not met.

Author: dogeared

impl/src/main/java/com/stormpath/sdk/impl/directory/OktaDirectory.java

@@ -246,6 +246,7 @@ public Schema getAccountSchema() {
 
     @SuppressWarnings("unchecked")
     private PasswordPolicy transformOktaPasswordPolicy(OktaPasswordPolicy oktaPasswordPolicy) {
+        // ref: http://developer.okta.com/docs/api/resources/policy.html#GroupPasswordPolicy
         final Map<String, Object> strengthMap = (Map<String, Object>)
             ((Map<String, Object>)oktaPasswordPolicy.getSettings().get("password")).get("complexity");
         PasswordPolicy ret = new PasswordPolicy() {

impl/src/main/java/com/stormpath/sdk/impl/ds/DefaultDataStore.java

@@ -31,6 +31,7 @@
 import com.stormpath.sdk.impl.ds.cache.ReadCacheFilter;
 import com.stormpath.sdk.impl.ds.cache.WriteCacheFilter;
 import com.stormpath.sdk.impl.error.DefaultError;
+import com.stormpath.sdk.impl.error.OktaError;
 import com.stormpath.sdk.impl.http.CanonicalUri;
 import com.stormpath.sdk.impl.http.HttpHeaders;
 import com.stormpath.sdk.impl.http.HttpHeadersHolder;
@@ -65,6 +66,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import javax.servlet.http.HttpServletResponse;
 import java.io.InputStream;
 import java.io.UnsupportedEncodingException;
 import java.net.URLEncoder;
@@ -105,6 +107,8 @@ public class DefaultDataStore implements InternalDataStore {
 
     private static final boolean COLLECTION_CACHING_ENABLED = false; //EXPERIMENTAL - set to true only while developing.
 
+    private static boolean oktaEnabled;
+
     private final RequestExecutor requestExecutor;
     private final ResourceFactory resourceFactory;
     private final MapMarshaller mapMarshaller;
@@ -160,6 +164,12 @@ public DefaultDataStore(RequestExecutor requestExecutor, BaseUrlResolver baseUrl
         this.cacheResolver = new DefaultCacheResolver(this.cacheManager, new DefaultCacheRegionNameResolver());
         this.apiKeyResolver = apiKeyResolver;
 
+        if (baseUrlResolver.getBaseUrl().toLowerCase().contains("okta")) {
+            oktaEnabled = true;
+        } else {
+            oktaEnabled = false;
+        }
+
         ReferenceFactory referenceFactory = new ReferenceFactory();
         this.resourceConverter = new DefaultResourceConverter(referenceFactory);
 
@@ -600,7 +610,15 @@ private Response execute(Request request) throws ResourceException {
                 body.put(DefaultError.REQUEST_ID.getName(), requestId);
             }
 
-            DefaultError error = new DefaultError(body);
+            com.stormpath.sdk.error.Error error;
+            if (oktaEnabled) {
+                OktaError oktaError = new OktaError(body);
+                // Okta Error response doesn't have status
+                oktaError.setProperty(OktaError.STATUS.getName(), response.getHttpStatus());
+                error = oktaError;
+            } else {
+                error = new DefaultError(body);
+            }
 
             throw new ResourceException(error);
         }

impl/src/main/java/com/stormpath/sdk/impl/error/OktaError.java

@@ -0,0 +1,92 @@
+/*
+ * Copyright 2013 Stormpath, Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.stormpath.sdk.impl.error;
+
+import com.stormpath.sdk.error.Error;
+import com.stormpath.sdk.impl.resource.AbstractResource;
+import com.stormpath.sdk.impl.resource.IntegerProperty;
+import com.stormpath.sdk.impl.resource.ListProperty;
+import com.stormpath.sdk.impl.resource.Property;
+import com.stormpath.sdk.impl.resource.StringProperty;
+
+import java.io.Serializable;
+import java.util.List;
+import java.util.Map;
+
+/**
+ * @since 0.1
+ */
+public class OktaError extends AbstractResource implements Error, Serializable {
+
+    static final long serialVersionUID = 42L;
+
+    public static final IntegerProperty STATUS = new IntegerProperty("status");
+    static final StringProperty ERROR_CODE = new StringProperty("errorCode");
+    static final StringProperty ERROR_SUMMARY = new StringProperty("errorSummary");
+    static final ListProperty ERROR_CAUSES = new ListProperty("errorCauses");
+    static final StringProperty ERROR_ID = new StringProperty("errorId");
+
+    private static final Map<String, Property> PROPERTY_DESCRIPTORS = createPropertyDescriptorMap(
+        STATUS, ERROR_CODE, ERROR_SUMMARY, ERROR_CAUSES, ERROR_ID
+    );
+
+    public OktaError(Map<String, Object> body) {
+        super(null, body);
+    }
+
+    // Needed for this class to be serializable
+    public OktaError() {
+        super(null, null);
+    }
+
+    @Override
+    public Map<String, Property> getPropertyDescriptors() {
+        return PROPERTY_DESCRIPTORS;
+    }
+
+    @Override
+    public int getStatus() {
+        return getInt(STATUS);
+    }
+
+    @Override
+    public int getCode() {
+        return 0;
+    }
+
+    @Override
+    @SuppressWarnings("unchecked")
+    public String getMessage() {
+        List causes = getListProperty(ERROR_CAUSES.getName());
+        return ((Map<String, String>)causes.get(0)).get("errorSummary");
+    }
+
+    @Override
+    public String getDeveloperMessage() {
+        return "";
+    }
+
+    @Override
+    public String getMoreInfo() {
+        return getString(ERROR_SUMMARY);
+    }
+
+    @Override
+    public String getRequestId() {
+       return getString(ERROR_ID);
+    }
+
+}