Redirect Handler throwOnMaxRedirect

[ronag]

Would like to have an option where the redirect handler throws on max redirects instead of just forwarding the result.

[ronag]

@metcoder95

[metcoder95]

It should be a matter of calling this.abort(new Error('max redirects') if the flag is set to true, right here:

undici/lib/handler/RedirectHandler.js

Lines 90 to 92 in ba4ca32

	this.location = this.history.length >= this.maxRedirections || util.isDisturbed(this.opts.body)
	? null
	: parseLocation(statusCode, headers)

Do you have something else to cover in mind?

[VigneshMurugan]

#2546

@metcoder95

[Lewiscowles1986]

See #951 as commented in #2260 the redirect interceptor and handler seem to lack any meaningful direct unit tests

[mertcanaltin]

landed: a7551ce

can I turn this issue off ?