fix: No image displayed when ImageResizeService throw an error - EXO-74337

commons-component-common/src/main/java/org/exoplatform/services/thumbnail/ImageResizeServiceImpl.java

@@ -76,16 +76,19 @@ public byte[] scaleImage(byte[] image, int width, int height, boolean fitExact,
       }
       bufferedImage = Scalr.resize(bufferedImage, resizeMethod, fitMode, width, height, Scalr.OP_ANTIALIAS);
     }
-
-
-    byte[] response = toByteArray(bufferedImage, imageReader);
-
-    if (!fitExact && response.length > image.length) {
-      // if the original image is smaller in weight from the resized image, we
-      // must keep the original image
+    try {
+      byte[] response = toByteArray(bufferedImage, imageReader);
+
+      if (!fitExact && response.length > image.length) {
+        // if the original image is smaller in weight from the resized image, we
+        // must keep the original image
+        return image;
+      } else {
+        return response;
+      }
+    } catch (IOException e) {
+      LOG.error("Unable to read image to resize, return original size",e);
       return image;
-    } else {
-      return response;
     }
   }
 

commons-component-common/src/test/java/org/exoplatform/services/thumbnail/ImageResizeServiceImplTest.java

@@ -195,4 +195,21 @@ public void testShouldFitHeight() {
       fail();
     }
   }
+
+  @Test
+  public void testResizeJpegWithNotSupportedMetadata() throws Exception {
+
+    File
+        fileImage =
+        new File(getClass().getClassLoader().getResource("images/illustration_with_non_supported_metadata.jpeg").getFile());
+    byte[] testFileImage = Files.readAllBytes(fileImage.toPath());
+    try {
+      byte[] resizedImage = imageResizeService.scaleImage(testFileImage, 250, 150, false, true);
+      assertEquals(resizedImage.length, testFileImage.length);
+    } catch (Exception e) {
+      fail("ScaleImage throws an error, and we do not catch it");
+    }
+
+  }
+
 }