Use InputStream.readAllBytes for DeleteObjects body

gaul · claude · gaul · commit 3fa8fb44b34c · 2026-04-16T15:24:26.000-07:00
Fixes modernizer violation introduced in e18583c. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
diff --git a/src/main/java/org/gaul/s3proxy/S3ProxyHandler.java b/src/main/java/org/gaul/s3proxy/S3ProxyHandler.java
@@ -1701,7 +1701,7 @@ private void handleMultiBlobRemove(HttpServletRequest request,
         if (expected.bits() != MD5.bits()) {
             throw new S3Exception(S3ErrorCode.INVALID_DIGEST);
         }
-        byte[] body = ByteStreams.toByteArray(is);
+        byte[] body = is.readAllBytes();
         HashCode actual = MD5.hashBytes(body);
         if (!expected.equals(actual)) {
             throw new S3Exception(S3ErrorCode.BAD_DIGEST);

Original file line number	Diff line number	Diff line change
`@@ -1701,7 +1701,7 @@ private void handleMultiBlobRemove(HttpServletRequest request,`
`1701`	`1701`	`if (expected.bits() != MD5.bits()) {`
`1702`	`1702`	`throw new S3Exception(S3ErrorCode.INVALID_DIGEST);`
`1703`	`1703`	`}`
`1704`		`- byte[] body = ByteStreams.toByteArray(is);`
	`1704`	`+ byte[] body = is.readAllBytes();`
`1705`	`1705`	`HashCode actual = MD5.hashBytes(body);`
`1706`	`1706`	`if (!expected.equals(actual)) {`
`1707`	`1707`	`throw new S3Exception(S3ErrorCode.BAD_DIGEST);`