http-debug enabled shouldn't exit the script on errors ... ever

[mstoykov]

Currently, if http-debug is set and the dumping of the request or response results in an error k6 will exit with status code 1 ... which is unlikely to be what anyone wants ...

[berndhartzer]

Is there some specific functionality needed to fix this, or is it just a matter of swapping Fatal() (which calls os.Exit(1)) with a different log level, such as Error() or Warn()?

[mstoykov]

No, it is just swapping it, I think. I don't know whether Error or Warn is better though 🤷‍♂️

You are welcome to make a PR, but we are currently working on PR 1007 (not linking it as it is huge as it is) and given that this bug has been this way for way too long, it seems unlikely we will once again stop working on 1007 to fix it, release it and then go back, as I've explained in other currently open PRs :(.

[berndhartzer]

Ok cool, thanks. Well I made a PR. In the end I thought Error() is a better choice because it is dumping out an error. Happy for any feedback of course!