Merge pull request #282 from AtlasOfLivingAustralia/feature/issue281

Support non-text response from postMultipart #281

Author: temi

grails-app/services/au/org/ala/ecodata/forms/EcpWebService.groovy

@@ -615,19 +615,19 @@ class EcpWebService {
 
         // Execute request
         try (Response response = client.newCall(requestBuilder.build()).execute()) {
-            okhttp3.MediaType respContentType = response.body().contentType()
-            String subType = respContentType.subtype()?.toLowerCase()
+
+            ResponseBody responseBody = response.body()
+            okhttp3.MediaType respContentType = responseBody.contentType()
+            String mediaType = respContentType.type()?.toLowerCase()
             def responseData
-            String responseBody = response.body().string()
-            switch (subType) {
-                case "json":
+            if (isTextBased(mediaType)) {
+                responseData = responseBody.string()
+                if (mediaType.contains('json')) {
                     ObjectMapper objectMapper = new ObjectMapper()
-                    responseData = objectMapper.readValue(responseBody, Object)
-                    break
-                case "text":
-                default:
-                    responseData = responseBody
-                    break
+                    responseData = objectMapper.readValue(responseData, Object)
+                }
+            } else {
+                responseData = responseBody.byteStream()
             }
 
             result.statusCode = result.status = response.code()
@@ -641,6 +641,18 @@ class EcpWebService {
         return result
     }
 
+    private static boolean isTextBased(String contentType) {
+        if (!contentType) {
+            log.error("Missing content type")
+            return false
+        }
+        String contentTypeLower = contentType.toLowerCase()
+        return contentTypeLower.contains('json') ||
+                contentTypeLower.contains('text') ||
+                contentTypeLower.contains('xml') ||
+                contentTypeLower.contains('graphql')
+    }
+
     /**
      * Forwards a HTTP multipart/form-data request to ecodata.
      * @param url the URL to forward to.
@@ -660,7 +672,7 @@ class EcpWebService {
     }
 
     Map postMultipart(String url, Map params, File file, String contentType, String originalFilename, String fileParamName = 'files', boolean useToken = false, boolean userToken = false) {
-        RequestBody fileBody = RequestBody.create(okhttp3.MediaType.parse(contentType), file)
+        RequestBody fileBody = RequestBody.create(file, okhttp3.MediaType.parse(contentType))
         postMultipart(url, params, fileBody, contentType, originalFilename, fileParamName, useToken, userToken)
     }