bugfix: Fix AUTO_WEBP handling.

Fixes aws-solutions#195
Fixes aws-solutions#200

Author: mik3y

CHANGELOG.md

@@ -8,4 +8,5 @@
 * Removed the additional `custom-resource` lambda.
 * Use `yarn`.
 * Use `eslint` and `prettier`.
-* Add hermetic build using Docker (`make build/serverless-image-handler.zip`)
+* Add hermetic build using Docker (`make release`)
+* Bugfix: Fixed handling of `AUTO_WEBP` env flag.

image-request.js

@@ -313,7 +313,7 @@ class ImageRequest {
    * @param {Object} event - The request body.
    */
   getOutputFormat(event) {
-    const autoWebP = process.env.AUTO_WEBP;
+    const autoWebP = process.env.AUTO_WEBP == "Yes";
     if (
       autoWebP &&
       event.headers.Accept &&

test/test-image-request.js

@@ -754,7 +754,7 @@ describe("getOutputFormat()", () => {
     it('Should pass if it returns "webp" for an accepts header which includes webp', () => {
       // Arrange
       process.env = {
-        AUTO_WEBP: true,
+        AUTO_WEBP: "Yes",
       };
       const event = {
         headers: {
@@ -773,7 +773,7 @@ describe("getOutputFormat()", () => {
     it("Should pass if it returns null for an accepts header which does not include webp", () => {
       // Arrange
       process.env = {
-        AUTO_WEBP: true,
+        AUTO_WEBP: "Yes",
       };
       const event = {
         headers: {
@@ -792,7 +792,7 @@ describe("getOutputFormat()", () => {
     it("Should pass if it returns null when AUTO_WEBP is disabled with accepts header including webp", () => {
       // Arrange
       process.env = {
-        AUTO_WEBP: false,
+        AUTO_WEBP: "No",
       };
       const event = {
         headers: {