♻️🎨 Improves error handling in the api-server

[pcrespov]

What do these changes do?

In this PR we do an initial refactoring by reorganizing the code base for errors in simcore_service_api_server. In this round we place all error logic under simcore_service_api_server.models.errors before doing changes in the logic

• ♻️ error handlers under simcore_service_api_server.exceptions
  • includes both exception handlers and exception classes ( also denoted error )
• 🎨 Common base class for all exceptions in this service ApiServerBaseError(OsparcErrorMixin, Exception)
• 🎨 Improved messages when unexpected exceptions ( closes API: Internal Server Error osparc-issues#491 )
  • human readable
  • OEC identifier
  • logs exception in the backend for diagnostics

Related issue/s

• closes API: improve error handling #4664
• related to API: Modify error and log responses in webserver rest API #443
• closes API: Internal Server Error osparc-issues#491

How to test

• Driving test for exception handlers in services/api-server/tests/unit/test_exceptions.py

Dev-ops checklist

• No ENV changes or I properly updated ENV (read the instruction)

[codecov]

Codecov Report

Attention: Patch coverage is 88.38384% with 23 lines in your changes are missing coverage. Please review.

Project coverage is 74.5%. Comparing base (cafbf96) to head (b52652e).
Report is 224 commits behind head on master.

❗ Current head b52652e differs from pull request most recent head ae8e58a

Please upload reports for the commit ae8e58a to get more accurate results.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5866      +/-   ##
=========================================
- Coverage    84.5%   74.5%   -10.1%     
=========================================
  Files          10     301     +291     
  Lines         214   14514   +14300     
  Branches       25     205     +180     
=========================================
+ Hits          181   10822   +10641     
- Misses         23    3640    +3617     
- Partials       10      52      +42     

Flag	Coverage Δ
integrationtests	71.3% <ø> (?)
unittests	89.2% <88.3%> (+4.6%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...src/simcore_service_api_server/api/routes/files.py	78.3% <100.0%> (ø)
...c/simcore_service_api_server/api/routes/solvers.py	65.0% <100.0%> (ø)
...core_service_api_server/api/routes/solvers_jobs.py	100.0% <100.0%> (ø)
...src/simcore_service_api_server/api/routes/users.py	100.0% <100.0%> (ø)
...c/simcore_service_api_server/api/routes/wallets.py	100.0% <100.0%> (ø)
...src/simcore_service_api_server/core/application.py	96.8% <100.0%> (ø)
.../simcore_service_api_server/exceptions/__init__.py	100.0% <100.0%> (ø)
...src/simcore_service_api_server/exceptions/_base.py	100.0% <100.0%> (ø)
...ore_service_api_server/exceptions/custom_errors.py	100.0% <100.0%> (ø)
...service_api_server/exceptions/handlers/__init__.py	100.0% <100.0%> (ø)
... and 16 more

... and 282 files with indirect coverage changes

[sanderegg]

nice and clean! thanks

[matusdrobuliak66]

Thanks!

[bisgaard-itis]

Very cool. Thanks a lot 🚀

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.2% Duplication on New Code

See analysis details on SonarCloud

[GitHK]

all good