When parsing data redundancy type fails, returns Unknown instead of exception.

Author: huiguangjun

src/main/java/com/aliyun/oss/internal/ResponseParsers.java

@@ -2724,10 +2724,12 @@ public static BucketInfo parseGetBucketInfo(InputStream responseBody) throws Res
 
             // owner
             Bucket bucket = new Bucket();
-            String id = bucketElem.getChild("Owner").getChildText("ID");
-            String displayName = bucketElem.getChild("Owner").getChildText("DisplayName");
-            Owner owner = new Owner(id, displayName);
-            bucket.setOwner(owner);
+            if (bucketElem.getChild("Owner") != null) {
+                String id = bucketElem.getChild("Owner").getChildText("ID");
+                String displayName = bucketElem.getChild("Owner").getChildText("DisplayName");
+                Owner owner = new Owner(id, displayName);
+                bucket.setOwner(owner);
+            }
 
             // bucket
             bucket.setName(bucketElem.getChildText("Name"));

src/main/java/com/aliyun/oss/model/DataRedundancyType.java

@@ -31,7 +31,12 @@ public enum DataRedundancyType {
     /**
      * ZRS
      */
-    ZRS("ZRS");
+    ZRS("ZRS"),
+
+    /**
+     * Unknown
+     */
+    Unknown("Unknown");
 
     private final String dataRedundancyTypeString;
 
@@ -60,6 +65,6 @@ public static DataRedundancyType parse(String dataRedundancyTypeString) {
             if (e.toString().equals(dataRedundancyTypeString))
                 return e;
         }
-        throw new IllegalArgumentException("Unsupported data redundancy type: " + dataRedundancyTypeString);
+        return Unknown;
     }
 }

src/test/java/com/aliyun/oss/common/parser/ResponseParsersTest.java

@@ -931,6 +931,8 @@ public void testParseGetBucketInfoBasic() {
         Assert.assertEquals(CannedAccessControlList.Private, result.getCannedACL());
         Assert.assertEquals("oss-cn-hangzhou", result.getBucket().getLocation());
         Assert.assertEquals("oss-example", result.getBucket().getName());
+        Assert.assertEquals("username", result.getBucket().getOwner().getDisplayName());
+        Assert.assertEquals("27183473914****", result.getBucket().getOwner().getId());
     }
 
     @Test
@@ -1022,6 +1024,41 @@ public void testParseGetBucketInfo() {
         Assert.assertEquals("xxx-id-123", result.getBucket().getResourceGroupId());
     }
 
+    @Test
+    public void testParseGetBucketInfoUnsupportType() {
+        String respBody = "" +
+                "<BucketInfo>\n" +
+                "  <Bucket>\n" +
+                "           <CreationDate>2013-07-31T10:56:21.000Z</CreationDate>\n" +
+                "            <ExtranetEndpoint>oss-cn-hangzhou.aliyuncs.com</ExtranetEndpoint>\n" +
+                "            <IntranetEndpoint>oss-cn-hangzhou-internal.aliyuncs.com</IntranetEndpoint>\n" +
+                "            <Location>oss-cn-hangzhou</Location>\n" +
+                "            <AccessControlList>\n" +
+                "              <Grant>private</Grant>\n" +
+                "            </AccessControlList>\n" +
+                "            <Comment>test</Comment>\n" +
+                "            <DataRedundancyType>ZRSII</DataRedundancyType>\n" +
+                "   </Bucket>\n" +
+                " </BucketInfo>";
+
+        InputStream instream = null;
+        try {
+            instream = new ByteArrayInputStream(respBody.getBytes("utf-8"));
+        } catch (UnsupportedEncodingException e) {
+            Assert.fail("UnsupportedEncodingException");
+        }
+
+        BucketInfo result = null;
+        try {
+            result = ResponseParsers.parseGetBucketInfo(instream);
+        } catch (ResponseParseException e) {
+            Assert.fail("parse bucket replication response body fail!");
+        }
+
+        Assert.assertEquals(DataRedundancyType.Unknown, result.getDataRedundancyType());
+
+    }
+
     @Test
     public void testParseListVpcip() {
         String respBody = "" +