From 07f49d034cef5f77e5d03d15f472e3b0c2bf8417 Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Mon, 1 Dec 2025 17:19:04 -0800 Subject: [PATCH] Add warning log when multiple checksum headers found in trailer - Log a warning when multiple valid checksum headers appear in trailers - Uses last checksum header as suggested by CodeRabbit reviewer - Improves debugging for edge cases with multiple checksum algorithms --- weed/s3api/chunked_reader_v4.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/weed/s3api/chunked_reader_v4.go b/weed/s3api/chunked_reader_v4.go index 52d7007aa..364bd57f7 100644 --- a/weed/s3api/chunked_reader_v4.go +++ b/weed/s3api/chunked_reader_v4.go @@ -530,6 +530,9 @@ func parseChunkChecksum(b *bufio.Reader) (ChecksumAlgorithm, []byte, error) { key := string(parts[0]) value := trimTrailingWhitespace(parts[1]) if alg, err := extractChecksumAlgorithm(key); err == nil { + if checksumAlgorithm != ChecksumAlgorithmNone { + glog.V(3).Infof("multiple checksum headers found in trailer, using last: %s", key) + } checksumAlgorithm = alg checksum = value }