HTTPResponseError code in "Handling client and server errors" section is incorrect.

[craigklem]

Sample code provided for "Handling client and server errors" section on main page is invalid. See: https://github.com/node-fetch/node-fetch#handling-client-and-server-errors

class HTTPResponseError extends Error {
	constructor(response, ...args) {
		this.response = response;
		super(`HTTP Error Response: ${response.status} ${response.statusText}`, ...args);
	}
}

1. trying to use code will fail with runtime exception. WebStorm shows problem: 'this' is not allowed before superclass constructor invocation.
2. ..args should be removed, node Error constructor only supports a single string argument. See: https://nodejs.org/docs/latest-v10.x/api/errors.html#errors_new_error_message

Reproduction

Steps to reproduce the behavior:

1. Copy and use sample code from "Handling client and server errors" section.

Expected behavior
Modified code that works should be like:

class HTTPResponseError extends Error {
	constructor(response) {
		super(`HTTP Error Response: ${response.status} ${response.statusText}`);
		this.response = response;
	}
}

Screenshots

Your Environment

software	version
node-fetch	2.6.1
node	14.15.4
npm	6.14.11
Operating System	Windows 10

Additional context

[dhananjaysa92]

looking into this issue

[dhananjaysa92]

PR created - #1666