fix(webp): better handling of error for page too big for webp

Belphemur · Aug 28, 2024 · a63d239 · a63d239
1 parent 839ad9e
commit a63d239
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/converter/webp/webp_converter.go b/converter/webp/webp_converter.go
@@ -17,6 +17,8 @@ import (
 	"sync/atomic"
 )
 
+const webpMaxHeight = 16383
+
 type Converter struct {
 	maxHeight  int
 	cropHeight int
@@ -202,6 +204,9 @@ func (converter *Converter) checkPageNeedsSplit(page *manga.Page) (bool, image.I
 	bounds := img.Bounds()
 	height := bounds.Dy()
 
+	if height >= converter.maxHeight {
+		return false, img, format, fmt.Errorf("page[%d] height %d exceeds maximum height %d of webp format", page.Index, height, converter.maxHeight)
+	}
 	return height >= converter.maxHeight, img, format, nil
 }