go-goyave / goyave / 33468163319

97%

master: 97%

Pull #293

github

fix: don't double-panic in recovery middleware on empty error slices

errors.NewSkip returns nil for nil or empty-slice panic values, so
the unconditional type assertion panicked again inside the deferred
recovery, breaking the connection instead of returning a 500. Use a
comma-ok assertion and add a regression test for panic([]error{}).

4 of 4 new or added lines in 1 file covered. (100.0%)

6712 of 6883 relevant lines covered (97.52%)

298.03 hits per line