Merge pull request #174 from IBM/Albert-Master-New

issue #170 miss charset in content-type of 400 response
LinuxForHealth · Oct 1, 2019 · 18e9e00 · 18e9e00
2 parents fbb9c90 + fb8be1b
commit 18e9e00
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/fhir-provider/src/main/java/com/ibm/fhir/provider/util/FHIRProviderUtil.java b/fhir-provider/src/main/java/com/ibm/fhir/provider/util/FHIRProviderUtil.java
@@ -27,15 +27,24 @@ public static MediaType getMediaType(String acceptHeader) {
             // ignore
         }
         if (mediaType != null) {
-            if (mediaType.isCompatible(FHIRMediaType.APPLICATION_FHIR_JSON_TYPE)) {
-                return FHIRMediaType.APPLICATION_FHIR_JSON_TYPE;
+            MediaType outMediaType = null;
+            if (mediaType.isCompatible(FHIRMediaType.APPLICATION_FHIR_JSON_TYPE)) {               
+                outMediaType = FHIRMediaType.APPLICATION_FHIR_JSON_TYPE;
             } else if (mediaType.isCompatible(FHIRMediaType.APPLICATION_JSON_TYPE)) {
-                return MediaType.APPLICATION_JSON_TYPE;
+                outMediaType = MediaType.APPLICATION_JSON_TYPE;
             } else if (mediaType.isCompatible(FHIRMediaType.APPLICATION_FHIR_XML_TYPE)) {
-                return FHIRMediaType.APPLICATION_FHIR_XML_TYPE;
+                outMediaType = FHIRMediaType.APPLICATION_FHIR_XML_TYPE;
             } else if (mediaType.isCompatible(FHIRMediaType.APPLICATION_XML_TYPE)) {
-                return MediaType.APPLICATION_XML_TYPE;
+                outMediaType = MediaType.APPLICATION_XML_TYPE;
+            } else {
+                outMediaType = FHIRMediaType.APPLICATION_FHIR_JSON_TYPE;
             }
+            // Need to get the charset setting from the acceptHeader if there
+            if (mediaType.getParameters() != null
+                    && mediaType.getParameters().get("charset") != null) {
+                outMediaType = outMediaType.withCharset(mediaType.getParameters().get("charset"));
+            }
+            return outMediaType;
         }
         // default
         return FHIRMediaType.APPLICATION_FHIR_JSON_TYPE;