Merge pull request #283 from AtlasOfLivingAustralia/feature/issue281

Feature/issue281

Author: temi

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

@@ -618,7 +618,7 @@ class EcpWebService {
 
             ResponseBody responseBody = response.body()
             okhttp3.MediaType respContentType = responseBody.contentType()
-            String mediaType = respContentType.type()?.toLowerCase()
+            String mediaType = respContentType.toString()?.toLowerCase()
             def responseData
             if (isTextBased(mediaType)) {
                 responseData = responseBody.string()
@@ -627,7 +627,10 @@ class EcpWebService {
                     responseData = objectMapper.readValue(responseData, Object)
                 }
             } else {
-                responseData = responseBody.byteStream()
+                // This is only used for the /document/createThumbnail endpoint.  Because the thumbnail is small we
+                // can get away with reading the entire response into memory.  If dealing with large files
+                // we might have to remove the try with resources and handle the stream manually.
+                responseData = responseBody.bytes()
             }
 
             result.statusCode = result.status = response.code()