Skip to content

Commit

Permalink
Merge pull request #627 from cvubrugier/issue-626-multipart-remove-parts
Browse files Browse the repository at this point in the history
fix: fix removal of the upload directory in posix CompleteMultipartUpload
  • Loading branch information
benmcclelland authored Jun 12, 2024
2 parents c6e1757 + db314a4 commit 1b922ca
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion backend/posix/posix.go
Original file line number Diff line number Diff line change
Expand Up @@ -656,7 +656,7 @@ func (p *Posix) CompleteMultipartUpload(ctx context.Context, input *s3.CompleteM
}

// cleanup tmp dirs
os.RemoveAll(upiddir)
os.RemoveAll(filepath.Join(bucket, objdir, uploadID))
// use Remove for objdir in case there are still other uploads
// for same object name outstanding, this will fail if there are
os.Remove(filepath.Join(bucket, objdir))
Expand Down

0 comments on commit 1b922ca

Please sign in to comment.