Don't emit end before error on Node.js <= 12

Fixes #1290

source/core/index.ts

@@ -1306,6 +1306,9 @@ export default class Request extends Duplex implements RequestEvents<Request> {
 			// Force clean-up, because some packages (e.g. nock) don't do this.
 			request.destroy();
 
+			// Node.js <= 12.18.2 mistakenly emits the response `end` first.
+			(request as any).res.removeAllListeners('end');
+
 			if (error instanceof TimedOutTimeoutError) {
 				error = new TimeoutError(error, this.timings!, this);
 			} else {

test/stream.ts

@@ -42,6 +42,10 @@ const headersHandler: Handler = (request, response) => {
 	response.end(JSON.stringify(request.headers));
 };
 
+const infiniteHandler: Handler = (_request, response) => {
+	response.write('foobar');
+};
+
 test('`options.responseType` is ignored', withServer, async (t, server, got) => {
 	server.get('/', defaultHandler);
 
@@ -410,3 +414,26 @@ test('async iterator works', withServer, async (t, server, got) => {
 
 	t.is(Buffer.concat(chunks).toString(), payload);
 });
+
+if (process.versions.node.split('.')[0] <= '12') {
+	test('does not emit end event on error', withServer, async (t, server, got) => {
+		server.get('/', infiniteHandler);
+
+		await t.notThrowsAsync(new Promise((resolve, reject) => {
+			got.stream({
+				timeout: 100,
+				hooks: {
+					beforeError: [
+						async error => {
+							await new Promise(resolve => setTimeout(resolve, 50));
+
+							return error;
+						}
+					]
+				}
+			}).once('end', () => {
+				reject('Stream has ended before erroring');
+			}).once('error', resolve).resume();
+		}));
+	});
+}