Performance improvement

Skip data parsing if charset is already given in the content-type header Fixes #305
node-fetch · Oct 2, 2024 · 6022b84 · 6022b84
1 parent 0fb2cbf
commit 6022b84
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/source/utils/get-charset.ts b/source/utils/get-charset.ts
@@ -19,11 +19,15 @@ export default function getCharset(content: Buffer, headers?: Headers) {
 		charset = parseContentType(contentType);
 	}
 
+	if (charset) {
+		return charset;
+	}
+
 	// No charset in content type, peek at response body for at most 1024 bytes
 	const data = content.slice(0, 1024).toString();
 
 	// HTML5, HTML4 and XML
-	if (!charset && data) {
+	if (data) {
 		const $ = load(data);
 
 		charset = parseContentType(