Fixed WebP resolution limit

lamka02sk · Nov 26, 2024 · 8cb25eb · 8cb25eb
1 parent c3f0ece
commit 8cb25eb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "picturium"
-version = "0.1.2"
+version = "0.1.3"
 edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

diff --git a/src/services/formats.rs b/src/services/formats.rs
@@ -6,7 +6,7 @@ use libvips::VipsImage;
 use log::{error, warn};
 use crate::parameters::format::Format;
 use crate::parameters::UrlParameters;
-use crate::pipeline::{PipelineError, PipelineResult};
+use crate::pipeline::{PipelineError, PipelineResult};   
 
 const WEBP_MAX_WIDTH: i32 = 16383; // px
 const WEBP_MAX_HEIGHT: i32 = 16383; // px
@@ -124,7 +124,7 @@ pub fn validate_output_format(image: &VipsImage, url_parameters: &UrlParameters<
     match output_format {
         OutputFormat::Webp => {
             let (width, height) = (image.get_width(), image.get_height());
-            let downsize = width > WEBP_MAX_WIDTH || height > WEBP_MAX_HEIGHT || (width * height) as f64 > WEBP_MAX_RESOLUTION;
+            let downsize = width > WEBP_MAX_WIDTH || height > WEBP_MAX_HEIGHT || (width * height) as f64 > (WEBP_MAX_RESOLUTION * 1_000_000.0);
 
             if !downsize {
                 return Ok(output_format.clone());