fix: putBucketLifecycle add ColdArchive and DeepColdArchive

README.md

@@ -957,12 +957,12 @@ parameters:
     - [createdBeforeDate] {String} expire date, e.g.: `2022-10-11T00:00:00.000Z`
       `createdBeforeDate` and `days` must have one.
   - [transition] {Object} Specifies the time when an object is converted to the IA or archive storage class during a valid life cycle.
-    - storageClass {String} Specifies the storage class that objects that conform to the rule are converted into. allow values: `IA` or `Archive`
+    - storageClass {String} Specifies the storage class that objects that conform to the rule are converted into. allow values: `IA` or `Archive` or `ColdArchive` or `DeepColdArchive`
     - [days] {Number|String} expire after the `days`
     - [createdBeforeDate] {String} expire date, e.g.: `2022-10-11T00:00:00.000Z`
       `createdBeforeDate` and `days` must have one.
   - [noncurrentVersionTransition] {Object} Specifies the time when an object is converted to the IA or archive storage class during a valid life cycle.
-    - storageClass {String} Specifies the storage class that history objects that conform to the rule are converted into. allow values: `IA` or `Archive`
+    - storageClass {String} Specifies the storage class that history objects that conform to the rule are converted into. allow values: `IA` or `Archive` or `ColdArchive` or `DeepColdArchive`
     - noncurrentDays {String} expire after the `noncurrentDays`
       `expiration`、 `abortMultipartUpload`、 `transition`、 `noncurrentVersionTransition` must have one.
   - [noncurrentVersionExpiration] {Object} specifies the expiration attribute of the lifecycle rules for the history object.

lib/common/bucket/putBucketLifecycle.js

@@ -73,6 +73,15 @@ function checkDaysAndDate(obj, key) {
   }
 }
 
+function checkNoncurrentDays(obj, key) {
+  const { noncurrentDays } = obj;
+  if (!noncurrentDays) {
+    throw new Error(`${key} must includes noncurrentDays`);
+  } else if (noncurrentDays && !/^[1-9][0-9]*$/.test(noncurrentDays)) {
+    throw new Error('noncurrentDays must be a positive integer');
+  }
+}
+
 function handleCheckTag(tag) {
   if (!isArray(tag) && !isObject(tag)) {
     throw new Error('tag must be Object or Array');
@@ -95,8 +104,8 @@ function checkRule(rule) {
   if (!['Enabled', 'Disabled'].includes(rule.status)) throw new Error('Status must be  Enabled or Disabled');
 
   if (rule.transition) {
-    if (!['IA', 'Archive'].includes(rule.transition.storageClass))
-      throw new Error('StorageClass must be  IA or Archive');
+    if (!['IA', 'Archive', 'ColdArchive', 'DeepColdArchive'].includes(rule.transition.storageClass))
+      throw new Error('StorageClass must be  IA、Archive、ColdArchive、DeepColdArchive');
     checkDaysAndDate(rule.transition, 'Transition');
   }
 
@@ -118,6 +127,12 @@ function checkRule(rule) {
     );
   }
 
+  if (rule.noncurrentVersionTransition) {
+    if (!['IA', 'Archive', 'ColdArchive', 'DeepColdArchive'].includes(rule.noncurrentVersionTransition.storageClass))
+      throw new Error('noncurrentVersionTransition StorageClass must be  IA、Archive、ColdArchive、DeepColdArchive');
+    checkNoncurrentDays(rule.noncurrentVersionTransition, 'NoncurrentVersionTransition');
+  }
+
   if (rule.tag) {
     if (rule.abortMultipartUpload) {
       throw new Error('Tag cannot be used with abortMultipartUpload');

test/config.js

@@ -1,7 +1,7 @@
 const { env } = process;
 
 const config = module.exports;
-const USWEST = 'oss-us-west-1'; // ONCI=true Using the region of Silicon Valley in the United States would be faster
+const USWEST = 'oss-ap-southeast-1'; // ONCI=true Faster using oss-ap-outsoutheast-1
 
 config.oss = {
   accessKeyId: env.ALI_SDK_OSS_ID,

test/node/bucket.test.js

@@ -857,6 +857,39 @@ describe('test/bucket.test.js', () => {
         }
       ]);
       assert.equal(putresult2.res.status, 200);
+      const putresult3 = await store.putBucketLifecycle(bucket, [
+        {
+          id: 'transition3',
+          prefix: 'logs/',
+          status: 'Enabled',
+          transition: {
+            days: 20,
+            storageClass: 'ColdArchive'
+          },
+          tag: {
+            key: 'test3',
+            value: '123'
+          }
+        }
+      ]);
+      assert.equal(putresult3.res.status, 200);
+      // Regions that need to support DeepColdArchive
+      const putresult4 = await store.putBucketLifecycle(bucket, [
+        {
+          id: 'transition4',
+          prefix: 'logs/',
+          status: 'Enabled',
+          transition: {
+            days: 20,
+            storageClass: 'DeepColdArchive'
+          },
+          tag: {
+            key: 'test4',
+            value: '123'
+          }
+        }
+      ]);
+      assert.equal(putresult4.res.status, 200);
     });
 
     it('should put the lifecycle with expiration and Tag', async () => {
@@ -987,7 +1020,7 @@ describe('test/bucket.test.js', () => {
         ]);
         assert(false);
       } catch (error) {
-        assert(error.message.includes('IA or Archive'));
+        assert(error.message.includes('IA、Archive'));
       }
     });
 

test/node/multiversion.test.js

@@ -145,7 +145,7 @@ describe('test/multiversion.test.js', () => {
           expiration: {
             expiredObjectDeleteMarker: 'true'
           },
-          NoncurrentVersionExpiration: {
+          noncurrentVersionExpiration: {
             noncurrentDays: 1
           }
         }
@@ -156,24 +156,49 @@ describe('test/multiversion.test.js', () => {
     });
 
     it('should putBucketLifecycle with noncurrentVersionTransition', async () => {
-      const putresult1 = await store.putBucketLifecycle(bucket, [
+      const putresult = await store.putBucketLifecycle(bucket, [
         {
-          id: 'expiration1',
-          prefix: 'logs/',
+          prefix: 'log/',
           status: 'Enabled',
           noncurrentVersionTransition: {
             noncurrentDays: '10',
             storageClass: 'IA'
           }
+        },
+        {
+          prefix: 'logs/',
+          status: 'Enabled',
+          noncurrentVersionTransition: {
+            noncurrentDays: '10',
+            storageClass: 'Archive'
+          }
+        },
+        {
+          prefix: 'logss/',
+          status: 'Enabled',
+          noncurrentVersionTransition: {
+            noncurrentDays: '10',
+            storageClass: 'ColdArchive'
+          }
+        },
+        {
+          prefix: 'logsss/',
+          status: 'Enabled',
+          noncurrentVersionTransition: {
+            noncurrentDays: '10',
+            storageClass: 'DeepColdArchive'
+          }
         }
       ]);
-      assert.equal(putresult1.res.status, 200);
+      assert.equal(putresult.res.status, 200);
+
       const { rules } = await store.getBucketLifecycle(bucket);
       const [
         {
           noncurrentVersionTransition: { noncurrentDays, storageClass }
         }
       ] = rules;
+
       assert(noncurrentDays === '10' && storageClass === 'IA');
     });
   });