add more validation for lifecycle

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

Author: tangledbytes

src/endpoint/s3/ops/s3_put_bucket_lifecycle.js

@@ -140,7 +140,14 @@ async function put_bucket_lifecycle(req) {
             current_rule.abort_incomplete_multipart_upload = _.omitBy({
                 days_after_initiation: parse_lifecycle_field(rule.AbortIncompleteMultipartUpload[0].DaysAfterInitiation),
             }, _.isUndefined);
-            reject_empty_field(current_rule.abort_incomplete_multipart_upload);
+            reject_empty_field(current_rule.abort_incomplete_multipart_upload?.days_after_initiation);
+
+            if (current_rule.abort_incomplete_multipart_upload?.days_after_initiation < 1) {
+                throw new S3Error({
+                    ...S3Error.InvalidArgument,
+                    detail: 'when calling the PutBucketLifecycleConfiguration operation: \'DaysAfterInitiation\' for AbortIncompleteMultipartUpload action must be a positive integer',
+                });
+            }
         }
 
         if (rule.Transition?.length === 1) {
@@ -157,7 +164,7 @@ async function put_bucket_lifecycle(req) {
                 noncurrent_days: parse_lifecycle_field(rule.NoncurrentVersionExpiration[0].NoncurrentDays),
                 newer_noncurrent_versions: parse_lifecycle_field(rule.NoncurrentVersionExpiration[0].NewerNoncurrentVersions),
             }, _.isUndefined);
-            reject_empty_field(current_rule.noncurrent_version_expiration);
+            reject_empty_field(current_rule.noncurrent_version_expiration?.noncurrent_days);
         }
 
         if (rule.NoncurrentVersionTransition?.length === 1) {