Proposal: forwardErrors should default to true

[jagregory]

I just spent a few hours debugging why my client wasn't receiving any errors from my server anymore. Turns out it's because I put this middleware in and didn't enable the forwardErrors setting, that meant my errors were being silently swallowed and changing the behaviour of my application.

IMO, this is unintuitive behaviour, I expected errors to be reported to Sentry and my application to continue behaving the same way it was before.

I'm happy for you to close this if you disagree, but I just thought I'd mention it as it was surprising behaviour.

[tonyfromundefined]

I had exactly the same situation. I agree with this proposal.

[Undistraction]

Definitely agree with this. It is a very unexpected default.

[BrunoScheufler]

Yeah that is absolutely true, I fully agree 👍 The reporting should not get in your way of handling the actual errors.