Issue #1627 - treat HEAD same as GET

Signed-off-by: Mike Schroeder <mschroed@us.ibm.com>
LinuxForHealth · May 10, 2021 · 0488fe7 · 0488fe7
1 parent db38cfb
commit 0488fe7
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/fhir-server/src/main/java/com/ibm/fhir/server/util/FHIRRestHelper.java b/fhir-server/src/main/java/com/ibm/fhir/server/util/FHIRRestHelper.java
@@ -1659,15 +1659,15 @@ private List<Entry> processEntriesByMethod(Bundle requestBundle, Map<Integer, En
             }
 
             // If we're working on a 'transaction' type interaction, or an interaction
-            // where we're processing only GET requests, then start a new transaction now.
+            // where we're processing only GET or HEAD requests, then start a new transaction now.
             BundleType.Value bundleType = requestBundle.getType().getValueAsEnum();
             if (bundleType == BundleType.Value.TRANSACTION ||
-                    (!requestEntriesByMethod.get(HTTPVerb.Value.GET).isEmpty() &&
+                    ((!requestEntriesByMethod.get(HTTPVerb.Value.GET).isEmpty() ||
+                            !requestEntriesByMethod.get(HTTPVerb.Value.HEAD).isEmpty()) &&
                             requestEntriesByMethod.get(HTTPVerb.Value.DELETE).isEmpty() &&
                             requestEntriesByMethod.get(HTTPVerb.Value.POST).isEmpty() &&
                             requestEntriesByMethod.get(HTTPVerb.Value.PUT).isEmpty() &&
-                            requestEntriesByMethod.get(HTTPVerb.Value.PATCH).isEmpty() &&
-                            requestEntriesByMethod.get(HTTPVerb.Value.HEAD).isEmpty())) {
+                            requestEntriesByMethod.get(HTTPVerb.Value.PATCH).isEmpty())) {
                 txn = new FHIRTransactionHelper(getTransaction());
                 txn.begin();
                 if (log.isLoggable(Level.FINE)) {