return first if response is already committed in DefaultHTTPErrorHandler

[mdaliyan]

I saw DefaultHTTPErrorHandler prepares an error response map and then discards everything if the response was already committed.

It can return earlier and do nothing if the response was already committed.

More readable and one less nesting.

[codecov]

Codecov Report

Merging #1956 (d27ec40) into master (499097e) will increase coverage by 0.03%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##           master    #1956      +/-   ##
==========================================
+ Coverage   90.99%   91.03%   +0.03%     
==========================================
  Files          32       32              
  Lines        2799     2800       +1     
==========================================
+ Hits         2547     2549       +2     
+ Misses        161      160       -1     
  Partials       91       91              

Impacted Files	Coverage Δ
echo.go	94.15% <37.50%> (+0.32%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 499097e...d27ec40. Read the comment docs.

[mdaliyan]

There's no change in the DefaultHTTPErrorHandler function behavior. Newly added tests will pass in the master branch and this branch both.

ps: the coverage is now higher than before. I don't know how to tell Codecov to refresh the report.

[aldas]

LGTM