mcollina / autocannon

fast HTTP/1.1 benchmarking tool written in Node.js
MIT License
7.76k stars 322 forks source link

Forever option fails from second iteration onwards #482

Open LRagji opened 1 year ago

LRagji commented 1 year ago

It seem like when you run with forever option the first iteration works perfectly fine but the next ones just gets sort-circuited and cancelled cause of this line which in my debugging opinion should be like this line which executes all iteration correctly....

again I am no expert on this repo, so excuse me if I am wrong

LRagji commented 1 year ago

There is one other change we have to add this line numRunning = opts.connections to restart function at line 171 cause at line 280 which is identifies running connections is decrementing it so a restart will need to reset this variable...

I think we need more testing with permutation combinations for forever option.