Not handling null 'result' value in statusCallback

[TimNZ]

loadtest/lib/pool.js

Line 75 in a0f4b30

result.requestIndex = this.requestIndex++

	finishRequest(client, result, error) {
		if (this.options.statusCallback) {
			result.requestIndex = this.requestIndex++
			result.instanceIndex = this.loadTest.instanceIndex
			this.options.statusCallback(error, result);

In my case 'error' is 'Connection error: connect ECONNREFUSED 127.0.0.1:3007', and result is null,
resulting in 'TypeError: Cannot set properties of undefined (setting 'requestIndex')' on setting result.requestIndex

[alexfernandez]

Hi @TimNZ , sorry for the delay. Good catch, care to send a patch?

[lagouyn]

Similar issue here:

In HttpClient.finishRequest(), on an ECONNREFUSED, result is undefined, so for its call to Pool.finishRequest() this crashes:

result.requestIndex = this.requestIndex++

[alexfernandez]

Hi @lagouyn , care to send a patch?