Merge pull request #1255 from versity/ben/fix-scoutfs-mp-etag-check

fix: scoutfs etag check for multipart uploads
This commit is contained in:
Ben McClelland
2025-05-02 12:03:08 -07:00
committed by GitHub
+5 -1
View File
@@ -287,7 +287,7 @@ func (s *ScoutFS) CompleteMultipartUpload(ctx context.Context, input *s3.Complet
if err != nil {
etag = ""
}
if parts[i].ETag == nil || etag != *parts[i].ETag {
if parts[i].ETag == nil || !areEtagsSame(etag, *parts[i].ETag) {
return res, "", s3err.GetAPIError(s3err.ErrInvalidPart)
}
@@ -1025,3 +1025,7 @@ func isNoAttr(err error) bool {
}
return false
}
func areEtagsSame(e1, e2 string) bool {
return strings.Trim(e1, `"`) == strings.Trim(e2, `"`)
}