From 3fc0ac44303df17a4d20610127b75fe6ab94d72f Mon Sep 17 00:00:00 2001 From: chrislu Date: Wed, 12 Nov 2025 17:25:23 -0800 Subject: [PATCH] validate --- weed/s3api/s3api_bucket_policy_engine.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/weed/s3api/s3api_bucket_policy_engine.go b/weed/s3api/s3api_bucket_policy_engine.go index 4a3a5fe4a..f4be80056 100644 --- a/weed/s3api/s3api_bucket_policy_engine.go +++ b/weed/s3api/s3api_bucket_policy_engine.go @@ -91,6 +91,14 @@ func (bpe *BucketPolicyEngine) DeleteBucketPolicy(bucket string) error { // - evaluated: whether a policy was found and evaluated (false = no policy exists) // - error: any error during evaluation func (bpe *BucketPolicyEngine) EvaluatePolicy(bucket, object, action, principal string) (allowed bool, evaluated bool, err error) { + // Validate required parameters + if bucket == "" { + return false, false, fmt.Errorf("bucket cannot be empty") + } + if action == "" { + return false, false, fmt.Errorf("action cannot be empty") + } + // Convert action to S3 action format s3Action := convertActionToS3Format(action)