Stop capturing panics and trying to continue execution after them. #3748
Labels
evaluation needed
proposal needs to be validated or tested before fully implementing it in k6
Milestone
What?
Drop the following code and it uses:
k6/js/common/util.go
Lines 67 to 87 in 153e0e5
Why?
The code was added many years ago when k6 had a bunch of really hard to fix race conditions. And was left as a safeguard.
Unfortunately it has two problems - it is not enough for most cases with multiple goroutines/async. And probably even worse - makes a bunch of potential bugs seem like less of an issue.
We have had at least one user come and ask us to have this be optional as it was problem during extension development.
Also, as far as I am aware it hasn't caught a single issue where k6 continuing has been a good idea for years. Some of the early things were nice, but now it seems like it just gets in the way.
Suggestion:
Just remove the code and it's called site to be rewritten with no defers.
The text was updated successfully, but these errors were encountered: