*: Add partial support for GetBucketLifecycle operation

Closes #1049. Signed-off-by: Evgenii Baidakov <[email protected]>
nspcc-dev · Jan 27, 2025 · e7921e2 · e7921e2
1 parent 2e8ec06
commit e7921e2
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 11 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@ This document outlines major changes between releases.
 ### Added
 
 ### Changed
+- GetBucketLifecycle operation now partially supported. It always returns NoSuchLifecycleConfiguration error (#1049)
 
 ### Fixed
 

diff --git a/api/handler/lifecycle.go b/api/handler/lifecycle.go
@@ -0,0 +1,17 @@
+package handler
+
+import (
+	"net/http"
+
+	"github.com/nspcc-dev/neofs-s3-gw/api"
+	"github.com/nspcc-dev/neofs-s3-gw/api/s3errors"
+)
+
+func (h *handler) GetBucketLifecycleHandler(w http.ResponseWriter, r *http.Request) {
+	h.logAndSendError(
+		w,
+		"The lifecycle configuration does not exist",
+		api.GetReqInfo(r.Context()),
+		s3errors.GetAPIError(s3errors.ErrNoSuchLifecycleConfiguration),
+	)
+}
diff --git a/api/handler/unimplemented.go b/api/handler/unimplemented.go
@@ -11,10 +11,6 @@ func (h *handler) SelectObjectContentHandler(w http.ResponseWriter, r *http.Requ
 	h.logAndSendError(w, "not implemented", api.GetReqInfo(r.Context()), s3errors.GetAPIError(s3errors.ErrNotImplemented))
 }
 
-func (h *handler) GetBucketLifecycleHandler(w http.ResponseWriter, r *http.Request) {
-	h.logAndSendError(w, "not implemented", api.GetReqInfo(r.Context()), s3errors.GetAPIError(s3errors.ErrNotImplemented))
-}
-
 func (h *handler) GetBucketEncryptionHandler(w http.ResponseWriter, r *http.Request) {
 	h.logAndSendError(w, "not implemented", api.GetReqInfo(r.Context()), s3errors.GetAPIError(s3errors.ErrNotImplemented))
 }

diff --git a/docs/aws_s3_compat.md b/docs/aws_s3_compat.md
@@ -237,13 +237,13 @@ See [Ownership](./aws_s3_compat.md#ownership-controls) section for details.
      
 ## Lifecycle
 
-|    | Method                          | Comments |
-|----|---------------------------------|----------|
-| 🔵 | DeleteBucketLifecycle           |          |
-| 🔵 | GetBucketLifecycle              |          |
-| 🔵 | GetBucketLifecycleConfiguration |          |
-| 🔵 | PutBucketLifecycle              |          |
-| 🔵 | PutBucketLifecycleConfiguration |          |
+|    | Method                          | Comments                                             |
+|----|---------------------------------|------------------------------------------------------|
+| 🔵 | DeleteBucketLifecycle           |                                                      |
+| 🟡 | GetBucketLifecycle              | It always returns NoSuchLifecycleConfiguration error |
+| 🔵 | GetBucketLifecycleConfiguration |                                                      |
+| 🔵 | PutBucketLifecycle              |                                                      |
+| 🔵 | PutBucketLifecycleConfiguration |                                                      |
 
 ## Logging