Dead code in makeRequestStreamAsync()?

[alxndrsn]

parseResponseError() throws rather than returns it's result at

minio-js/src/internal/xml-parser.ts

Lines 117 to 119 in d8c6721

	throw e
	}

This means that at

minio-js/src/internal/client.ts

Lines 739 to 741 in d8c6721

	const err = await xmlParsers.parseResponseError(response)
	this.logHTTP(reqOptions, response, err)
	throw err

, makeRequestStreamAsync() does not call this.logHTTP() at line 740.

I noticed this, as ?location requests do not seem to emit close events.

[prakashsvmx]

getBucketRegionAsync implementation is little different.. it automatically infers/fetches the region if possible from the response else falls back to default region.

[alxndrsn]

getBucketRegionAsync implementation is little different.. it automatically infers/fetches the region if possible from the response else falls back to default region.

Line 739 of src/internal/client.ts always throws, so line 740 is never executed. I have opened a PR at #1401 to try to demonstrate the problem more clearly.