Added a optional config variable to disable centralized error handler in recovery middleware

[Omkar-C]

Recovery Middleware calls the centralized Error Handler. This is a change to have a optional config variable to disable centralized error handler in recovery. If the centrailzed Error Handler is disabled, panic error caught, will be returned to upstream middleware.

[Omkar-C]

Should I pass back the error with closure, so upstream middlewares can handle ?

[Omkar-C]

Should I pass back the error with closure, so upstream middlewares can handle ?

I have returned back the error, seems like that should be the ideal behavior, also added corrected tests

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.85%. Comparing base (47844c9) to head (e368ae0).
⚠️ Report is 186 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2410   +/-   ##
=======================================
  Coverage   92.84%   92.85%           
=======================================
  Files          39       39           
  Lines        4503     4507    +4     
=======================================
+ Hits         4181     4185    +4     
  Misses        234      234           
  Partials       88       88           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[aldas]

LGTM

[aldas]

For history sake - not returning error by default bothers me as it would be more correct behavior. But this would make middleware behave completely different as it has done since 2015. These simple things can be problematic as it changes how already existing application middleware chain work. Probably 98% time this is not an issue - but for some it could be.

life is about making compromises.

p.s. just looked my v5 branch and there I already have made so that c.Errror is not used anymore and we return that recovered "error"