Skip to content

remove setupErrorHandler() which prints unnecessary logs for 404

Diwakar Singh Mauryarequested to merge
8-prints-long-stack-trace-in-error-log-when-a-non-existing-route-is-hit into master

What this PR does / why we need it:

Which issue this PR fixes:

Closes #8 (closed)

What changes did you make? (Give an overview):
This merge request removes setupErrorHandler(). The long error logs for 404 were getting printed from this function's body. The function itself does not does much in comparison to default expressjs default error handler for 404. That's why it is better to remove it.

Special notes for your reviewer:

If applicable:

  • documentation
  • unit tests
  • tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Stop printing long error logs for requests on non-existing routes.

Merge request reports