Merge pull request #552 from sbplat/main

fix: use JPEGFactory for jpeg
Stirling-Tools · Dec 24, 2023 · 914dd0a · 914dd0a
2 parents c681f48 + 344d116
commit 914dd0a
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/stirling/software/SPDF/utils/PdfUtils.java b/src/main/java/stirling/software/SPDF/utils/PdfUtils.java
@@ -266,7 +266,10 @@ public static byte[] imageToPdf(MultipartFile[] files, String fitOption, boolean
                 } else {
                     BufferedImage image = ImageIO.read(file.getInputStream());
                     BufferedImage convertedImage = ImageProcessingUtils.convertColorType(image, colorType);
-                    PDImageXObject pdImage = LosslessFactory.createFromImage(doc, convertedImage);
+                    // Use JPEGFactory if it's JPEG since JPEG is lossy
+                    PDImageXObject pdImage = (contentType != null && contentType.equals("image/jpeg"))
+                                           ? JPEGFactory.createFromImage(doc, convertedImage)
+                                           : LosslessFactory.createFromImage(doc, convertedImage);
                     addImageToDocument(doc, pdImage, fitOption, autoRotate);
                 }
             }