Fix parsing empty and zero width/height value

- Fix Sharp error when zero is given
- Remove hexadecimal width / height value support
- Add tests for empty value

test/resizeImage.test.js

@@ -42,6 +42,22 @@ describe("resizeImage", () => {
       const output = await resizeImage(input, "png", "png", 50);
       expect(output.equals(expectedOutput)).toBe(true);
     });
+    it("It can handle an empty string height", async () => {
+      const input = fs.readFileSync(__dirname + "/fixtures/pixelme.png");
+      const expectedOutput = fs.readFileSync(
+        __dirname + "/fixtures/pixelme50x.png"
+      );
+      const output = await resizeImage(input, "png", "png", "50", "");
+      expect(output.equals(expectedOutput)).toBe(true);
+    });
+    it("It can handle height 0", async () => {
+      const input = fs.readFileSync(__dirname + "/fixtures/pixelme.png");
+      const expectedOutput = fs.readFileSync(
+        __dirname + "/fixtures/pixelme50x.png"
+      );
+      const output = await resizeImage(input, "png", "png", "50", "0");
+      expect(output.equals(expectedOutput)).toBe(true);
+    });
   });
 
   describe("Given an image with Orientation header", () => {

util/resizeImage.js

@@ -29,11 +29,14 @@ function resizeImage(
 
   const resizeOptions = { fit: fit };
 
-  if (width) {
-    resizeOptions.width = parseInt(width);
+  const parsedWidth = width ? parseInt(width, 10) : 0;
+  if (parsedWidth > 0) {
+    resizeOptions.width = parsedWidth;
   }
-  if (height) {
-    resizeOptions.height = parseInt(height);
+
+  const parsedHeight = height ? parseInt(height, 10) : 0;
+  if (parsedHeight > 0) {
+    resizeOptions.height = parsedHeight;
   }
 
   const sharp = SHARP(body).withMetadata().resize(resizeOptions);