Added restriction to limit the ID length to 255 characters in bucket …

…lifecycle rules Signed-off-by: Aayush Chouhan <[email protected]>
noobaa · Jan 2, 2025 · 0fd422f · 0fd422f
1 parent 6a079ed
commit 0fd422f
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/endpoint/s3/ops/s3_put_bucket_lifecycle.js b/src/endpoint/s3/ops/s3_put_bucket_lifecycle.js
@@ -88,7 +88,12 @@ async function put_bucket_lifecycle(req) {
         };
 
         if (rule.ID?.length === 1) {
-            current_rule.id = rule.ID[0];
+            if (rule.ID[0].length > 255) {
+                dbg.error('Rule should not have ID length exceeding 255 characters', rule);
+                throw new S3Error(S3Error.InvalidArgument);
+            } else {
+                current_rule.id = rule.ID[0];
+            }
         } else {
             // Generate a random ID if missing
             current_rule.id = uuid();