Added SVG validation: width and height must be specified #2195 #2062

Author: liborm85

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Added SVG validation: width and height must be specified (in SVG string/element or `svg` node)
+
 ## 0.2.22 - 2026-01-07
 
 - Added object type validation for parameters in method `createPdf` and `createPdfKitDocument`

src/docMeasure.js

@@ -215,6 +215,15 @@ DocMeasure.prototype.measureSVG = function (node) {
 
 	node.font = this.styleStack.getProperty('font');
 
+	// SVG requires a defined width and height
+	if (!isNumber(node._width) && !isNumber(node._height)) {
+		throw new Error('SVG is missing defined width and height.');
+	} else if (!isNumber(node._width)) {
+		throw new Error('SVG is missing defined width.');
+	} else if (!isNumber(node._height)) {
+		throw new Error('SVG is missing defined height.');
+	}
+
 	// scale SVG based on final dimension
 	node.svg = this.svgMeasure.writeDimensions(node.svg, {
 		width: node._width,

src/helpers.js

@@ -5,7 +5,7 @@ function isString(variable) {
 }
 
 function isNumber(variable) {
-	return typeof variable === 'number' || variable instanceof Number;
+	return ((typeof variable === 'number') || (variable instanceof Number)) && !Number.isNaN(variable);
 }
 
 function isBoolean(variable) {