Add some default binary types and minor tweaks

Author: billrobertson42

aws-serverless-java-container-core/src/main/java/com/amazonaws/serverless/proxy/internal/servlet/AwsProxyHttpServletResponseWriter.java

@@ -58,6 +58,15 @@ public AwsProxyResponse writeResponse(AwsHttpServletResponse containerResponse,
     }
 
     private boolean isBinary(String contentType) {
-        return contentType == null ? false : LambdaContainerHandler.getContainerConfig().isBinaryContentType(contentType);
+        if(contentType != null) {
+            int semidx = contentType.indexOf(';');
+            if(semidx >= 0) {
+                return LambdaContainerHandler.getContainerConfig().isBinaryContentType(contentType.substring(0, semidx));
+            }
+            else {
+                LambdaContainerHandler.getContainerConfig().isBinaryContentType(contentType);
+            }
+        }
+        return false;
     }
 }

aws-serverless-java-container-core/src/main/java/com/amazonaws/serverless/proxy/model/ContainerConfig.java

@@ -23,6 +23,7 @@ public static ContainerConfig defaultConfig() {
         configuration.setUseStageAsServletContext(false);
         configuration.setValidFilePaths(DEFAULT_FILE_PATHS);
         configuration.setQueryStringCaseSensitive(false);
+        configuration.addBinaryContentTypes("application/octet-stream", "image/jpeg", "image/png", "image/gif");
 
         return configuration;
     }
@@ -223,20 +224,23 @@ public void setQueryStringCaseSensitive(boolean queryStringCaseSensitive) {
     }
 
     /**
-     * Configure specified content type as binary
+     * Configure specified content type(s) as binary
+     * @param contentTypes list of exact content types that will be considered as binary
      */
-    public void addBinaryContentType(String type) {
-        if(type != null) {
-            binaryContentTypes.add(type);
+    public void addBinaryContentTypes(String... contentTypes) {
+        if(contentTypes != null) {
+            for(String type: contentTypes) {
+                binaryContentTypes.add(type);
+            }
         }
     }
 
     /**
      * Determine if specified content type has been configured as binary
-     * @param type
+     * @param contentType content type to query
      * @return
      */
-    public boolean isBinaryContentType(String type) {
-        return binaryContentTypes.contains(type);
+    public boolean isBinaryContentType(String contentType) {
+        return contentType != null && binaryContentTypes.contains(contentType.trim());
     }
 }