Support cloudinary credentials URL that has an API_KEY but no API_SECRET

This is required for mobile applications that are not supposed to use the API_SECRET.

Author: tocker

cloudinary-core/src/main/java/com/cloudinary/Cloudinary.java

@@ -246,7 +246,9 @@ protected Map parseConfigUrl(String cloudinaryUrl) {
 		if (cloudinaryUri.getUserInfo() != null) {
 			String[] creds = cloudinaryUri.getUserInfo().split(":");
 			params.put("api_key", creds[0]);
-			params.put("api_secret", creds[1]);
+			if (creds.length > 1) {
+				params.put("api_secret", creds[1]);
+			}
 		}
 		params.put("private_cdn", !StringUtils.isEmpty(cloudinaryUri.getPath()));
 		params.put("secure_distribution", cloudinaryUri.getPath());

cloudinary-test-common/src/main/java/com/cloudinary/test/AbstractUploaderTest.java

@@ -18,18 +18,16 @@
 import java.util.zip.ZipInputStream;
 import java.net.*;
 
+import com.cloudinary.*;
+import org.cloudinary.json.JSONArray;
 import org.junit.Before;
 import org.junit.BeforeClass;
 import org.junit.AfterClass;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TestName;
 
-import com.cloudinary.ArchiveParams;
-import com.cloudinary.Cloudinary;
-import com.cloudinary.Coordinates;
-import com.cloudinary.ResponsiveBreakpoints;
-import com.cloudinary.Transformation;
+import com.cloudinary.ResponsiveBreakpoint;
 import com.cloudinary.utils.ObjectUtils;
 import com.cloudinary.utils.Rectangle;
 
@@ -438,12 +436,32 @@ public void testFilenameOption() throws Exception {
 
     @Test
     public void testResponsiveBreakpoints() throws Exception {
-    	Map result = cloudinary.uploader().upload(SRC_TEST_IMAGE, ObjectUtils.asMap("responsive_breakpoints", 
-    			new ResponsiveBreakpoints().maxImages(2).createDerived(false)
+        ResponsiveBreakpoint breakpoint = new ResponsiveBreakpoint().maxImages(2).createDerived(false);
+
+        // A single breakpoint
+        Map result = cloudinary.uploader().upload(SRC_TEST_IMAGE, ObjectUtils.asMap("responsive_breakpoints",
+                breakpoint
     		));
     	java.util.ArrayList breakpointsResponse = (java.util.ArrayList) result.get("responsive_breakpoints");
     	java.util.ArrayList breakpoints = (java.util.ArrayList)((Map) breakpointsResponse.get(0)).get("breakpoints");
     	assertEquals(2, breakpoints.size());
+
+        // an array of breakpoints
+    	result = cloudinary.uploader().upload(SRC_TEST_IMAGE, ObjectUtils.asMap("responsive_breakpoints",
+                new ResponsiveBreakpoint [] {breakpoint}
+    		));
+    	breakpointsResponse = (java.util.ArrayList) result.get("responsive_breakpoints");
+    	breakpoints = (java.util.ArrayList)((Map) breakpointsResponse.get(0)).get("breakpoints");
+    	assertEquals(2, breakpoints.size());
+
+        // a JSONArray of breakpoints
+        JSONArray array = new JSONArray();
+        array.put(breakpoint);
+    	result = cloudinary.uploader().upload(SRC_TEST_IMAGE, ObjectUtils.asMap("responsive_breakpoints", array
+    		));
+    	breakpointsResponse = (java.util.ArrayList) result.get("responsive_breakpoints");
+    	breakpoints = (java.util.ArrayList)((Map) breakpointsResponse.get(0)).get("breakpoints");
+    	assertEquals(2, breakpoints.size());
     }
 
 	@Test