Fix the HEAD request to avoid reading file for checksum digest

yma96 · Jun 26, 2024 · 54dae66 · 54dae66
1 parent 6631535
commit 54dae66
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/bindings/jaxrs/src/main/java/org/commonjava/indy/core/bind/jaxrs/ContentAccessHandler.java b/bindings/jaxrs/src/main/java/org/commonjava/indy/core/bind/jaxrs/ContentAccessHandler.java
@@ -380,13 +380,14 @@ public Response doHead( final String packageType, final String type, final Strin
 
                     responseHelper.setInfoHeaders( builder, item, sk, path, true, contentType,
                                     httpMetadata );
-
-                    if(!path.endsWith("/")) {
-                        // Content hashing headers
-                        builder.header(ApplicationHeader.md5.key(), contentDigester.digest(sk, path, new EventMetadata()).getDigests().get(ContentDigest.MD5));
-                        builder.header(ApplicationHeader.sha1.key(), contentDigester.digest(sk, path, new EventMetadata()).getDigests().get(ContentDigest.SHA_1));
-                    }
-
+
+                    // Remove this to avoid reading file for checksum digest during HEAD request
+//                    if(!path.endsWith("/")) {
+//                        // Content hashing headers
+//                        builder.header(ApplicationHeader.md5.key(), contentDigester.digest(sk, path, new EventMetadata()).getDigests().get(ContentDigest.MD5));
+//                        builder.header(ApplicationHeader.sha1.key(), contentDigester.digest(sk, path, new EventMetadata()).getDigests().get(ContentDigest.SHA_1));
+//                    }
+
                     if ( builderModifier != null )
                     {
                         builderModifier.accept( builder );