RFC 7807 Compliance

[jujaga]

Description

This PR standardizes how we emit errors to be RFC 7807 compliant, implements a thin validation middleware, standardizes problem handling, and various other minor fixes.

• 6329d2f Bugfix certain problem throws triggering unhandledRejection crashes
• bd5362b Refactor Problem handling to throw instead of sending
• c6d4a80 Revert lib-storage queueSize back to default of 4
• 8d72173 Ensure authentication happens first before validation
• 4cf7a1d Drop express-validation library
• e991f13 Implement new thin validation middleware
• 52f2b69 Ensure problem repsonses are RFC 7807 compliant

  SHOWCASE-3192

Types of changes

Bug fix (non-breaking change which fixes an issue) New feature (non-breaking change which adds functionality) Documentation (non-breaking change with enhancements to documentation) Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• [x] I have read the CONTRIBUTING doc
• [x] I have checked that unit tests pass locally with my changes
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] I have added necessary documentation (if appropriate)

Further comments

[codeclimate[bot]]

Code Climate has analyzed commit 6329d2ff and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Complexity	2
Duplication	6

The test coverage on the diff in this pull request is 84.4% (50% is the threshold).

This pull request will bring the total coverage in the repository to 64.7% (0.2% change).

View more on Code Climate.

[github-actions[bot]]

Coverage Report

Totals
Statements:	58% ( 2645 / 4560 )
Methods:	47.98% ( 297 / 619 )
Lines:	64.71% ( 1590 / 2457 )
Branches:	51.08% ( 758 / 1484 )