adjust fix
Some checks are pending
go: build dev binaries / cleanup (push) Waiting to run
go: build dev binaries / build_dev_linux_windows (amd64, linux) (push) Blocked by required conditions
go: build dev binaries / build_dev_linux_windows (amd64, windows) (push) Blocked by required conditions
go: build dev binaries / build_dev_darwin (amd64, darwin) (push) Blocked by required conditions
go: build dev binaries / build_dev_darwin (arm64, darwin) (push) Blocked by required conditions
docker: build dev containers / build-dev-containers (push) Waiting to run
End to End / FUSE Mount (push) Waiting to run
go: build binary / Build (push) Waiting to run
Ceph S3 tests / Ceph S3 tests (push) Waiting to run

fix https://github.com/seaweedfs/seaweedfs/issues/6497
This commit is contained in:
chrislu
2025-02-01 14:11:57 -08:00
parent a75271dd43
commit 42efade0dc
2 changed files with 7 additions and 4 deletions

View File

@@ -248,7 +248,7 @@ func (f *Filer) ensureParentDirectoryEntry(ctx context.Context, entry *Entry, di
} }
dirPath := "/" + util.Join(dirParts[:level]...) dirPath := "/" + util.Join(dirParts[:level]...)
// fmt.Printf("%d directory: %+v\n", i, dirPath) // fmt.Printf("%d dirPath: %+v\n", level, dirPath)
// check the store directly // check the store directly
glog.V(4).Infof("find uncached directory: %s", dirPath) glog.V(4).Infof("find uncached directory: %s", dirPath)
@@ -257,9 +257,11 @@ func (f *Filer) ensureParentDirectoryEntry(ctx context.Context, entry *Entry, di
// no such existing directory // no such existing directory
if dirEntry == nil { if dirEntry == nil {
if len(dirParts) >= 2 && level == 2 && dirParts[0] == "buckets" { // fmt.Printf("dirParts: %v %v %v\n", dirParts[0], dirParts[1], dirParts[2])
if err := s3bucket.VerifyS3BucketName(dirParts[1]); err != nil { // dirParts[0] == "" and dirParts[1] == "buckets"
return fmt.Errorf("invalid bucket name %s: %v", dirParts[1], err) if len(dirParts) >= 3 && dirParts[1] == "buckets" {
if err := s3bucket.VerifyS3BucketName(dirParts[2]); err != nil {
return fmt.Errorf("invalid bucket name %s: %v", dirParts[2], err)
} }
} }

View File

@@ -11,6 +11,7 @@ func Test_verifyBucketName(t *testing.T) {
"123.12.153.10", "123.12.153.10",
"abc214..2", "abc214..2",
"d", "d",
"aa",
".ewfs3253543", ".ewfs3253543",
"grehtrry-", "grehtrry-",
"----------", "----------",