mirrors/versitygw
1
0
Fork 0
mirror of https://github.com/versity/versitygw.git synced 2026-09-22 16:04:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2380 from versity/sis/posix-create-exsiting-bucket-err

Browse Source 
fix: return `BucketAlreadyExists` for existing dirs without acl meta attr
This commit is contained in:
Ben McClelland
2026-09-09 08:59:23 -07:00
committed by GitHub
parent 4e862bb7d9 a58e125184
commit 2c157f8974
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/posix/posix.go
+6
View File
@@ -669,6 +669,12 @@ func (p *Posix) CreateBucket(ctx context.Context, input *s3.CreateBucketInput, a
err = os.Mkdir(bucket, p.newDirPerm)
if err != nil && os.IsExist(err) {
aclJSON, err := p.meta.RetrieveAttribute(nil, bucket, "", aclkey)
if errors.Is(err, meta.ErrNoSuchKey) {
// The directory already exists but has no gateway-managed acl
// attribute, e.g. a preexisting directory on a dataset the
// gateway was pointed at rather than one created via CreateBucket
return s3err.GetBucketErr(s3err.ErrBucketAlreadyExists, bucket)
}
if err != nil {
return fmt.Errorf("get bucket acl: %w", err)
}