Don't return empty string for .json() if Transfer-Encoding is chunked (#464)

Author: davidrunger

source/core/Ky.ts

@@ -92,7 +92,8 @@ export class Ky {
 					}
 
 					const contentLength = response.headers.get('Content-Length');
-					if (contentLength === null || contentLength === '0') {
+					const transferEncoding = response.headers.get('Transfer-Encoding');
+					if ((contentLength === null || contentLength === '0') && transferEncoding !== 'chunked') {
 						return '';
 					}
 

test/main.ts

@@ -236,6 +236,22 @@ test('.json() with custom accept header', async t => {
 	await server.close();
 });
 
+test('.json() when response is chunked', async t => {
+	const server = await createHttpTestServer();
+	server.get('/', async (request, response) => {
+		response.write('[');
+		response.write('"one",');
+		response.write('"two"');
+		response.end(']');
+	});
+
+	const responseJson = await ky.get(server.url).json();
+
+	t.deepEqual(responseJson, ['one', 'two']);
+
+	await server.close();
+});
+
 test('.json() with invalid JSON body', async t => {
 	const server = await createHttpTestServer();
 	server.get('/', async (request, response) => {