Merge pull request #14 from aws-samples/dev

add support for HTTP compression

Author: bnusunny

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "aws-lambda-adapter"
-version = "0.1.0"
+version = "0.1.2"
 authors = ["Harold Sun <[email protected]>"]
 edition = "2018"
 

Cargo.toml

@@ -32,7 +32,7 @@ $ cd aws-lambda-adapter
 
 ### Compiling with Docker
 On x86_64 Windows, Linux and macOS, you can run one command to compile Lambda Adapter with docker.
-The Dockerfile is [here](Dockerfile.x86). [AWS CLI v2](https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2.html) should have been installed and configured.
+The Dockerfile is [here](Dockerfile). [AWS CLI v2](https://docs.aws.amazon.com/cli/latest/userguide/install-cliv2.html) should have been installed and configured.
 
 ```shell
 $ make build

README.md

@@ -30,7 +30,7 @@ http {
 
     keepalive_timeout  65;
 
-    #gzip  on;
+    gzip  on;
 
     include /etc/nginx/conf.d/*.conf;
 }

examples/nginx/app/config/nginx.conf

@@ -163,6 +163,7 @@ async fn http_proxy_handler(event: Request, _: Context) -> Result<impl IntoRespo
         serializer.finish();
     }
     debug!("app_url is {:#?}", app_url);
+    debug!("request headers are {:#?}", parts.headers);
 
     let app_response = HTTP_CLIENT
         .get()
@@ -186,18 +187,30 @@ async fn http_proxy_handler(event: Request, _: Context) -> Result<impl IntoRespo
 
 async fn convert_body(app_response: reqwest::Response) -> Result<Body, Error> {
     let content_type;
+    debug!("app response headers are {:#?}", app_response.headers());
+
+    if app_response
+        .headers()
+        .get(http::header::CONTENT_ENCODING)
+        .is_some()
+    {
+        debug!("body is binary");
+        let content = app_response.bytes().await?;
+        return Ok(Body::Binary(content.to_vec()));
+    }
+
     if let Some(value) = app_response.headers().get(http::header::CONTENT_TYPE) {
         content_type = value.to_str().unwrap_or_default();
     } else {
-        // default to "application/json" if content-type header is not available in the response
-        content_type = "application/json";
+        // default to "application/octet-stream" if content-type header is not available in the response
+        content_type = "application/octet-stream";
     }
+    debug!("content_type is {:?}", content_type);
 
     if content_type.starts_with("text")
-        || content_type.eq("application/json")
-        || content_type.eq("application/javascript")
-        || content_type.eq("application/xml")
-        || content_type.eq("image/svg+xml")
+        || content_type.starts_with("application/json")
+        || content_type.starts_with("application/javascript")
+        || content_type.starts_with("application/xml")
     {
         debug!("body is text");
         let body_text = app_response.text().await?;