🐛 Fix/storage skip invalid projects while listing

[pcrespov]

What do these changes do?

@drniiken could not copy a template project that he owned. This was a side effect of having corrupted data in both projects and meta_data tables. The copy routine was failing because a validation error on some data owned by the same user. The corrupt data had nothing to do with this operation but the error was not handled properly which resulted in a confusing 400 error with message=Invalid header token

Running the same request from storage i.e. GET http://storage:8080/v0/locations/0/files/metadata?user_id=5&uuid_filter=b7... revealed the validation error.

Two actions were taken:

1. fixed corrupted data in staging
2. create this PR that skips corrupted data in the database

This should prevent for the moment this error but does not completely solves the problem. Follow up should consider adding validation tests against deployed databases:

• Maintenance: audit/test validate deployed databases against current model-schemas #3739

[odeimaiz]

Thanks 💪

[codecov]

Codecov Report

Merging #3740 (df05c01) into master (ea8d620) will decrease coverage by 16.8%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #3740      +/-   ##
=========================================
- Coverage    79.8%   62.9%   -16.9%     
=========================================
  Files         776     264     -512     
  Lines       32125   13377   -18748     
  Branches      791       0     -791     
=========================================
- Hits        25636    8424   -17212     
+ Misses       6291    4953    -1338     
+ Partials      198       0     -198     

Flag	Coverage Δ
integrationtests	58.8% <ø> (?)
unittests	90.4% <100.0%> (+10.6%)	⬆️

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

Impacted Files	Coverage Δ
...storage/src/simcore_service_storage/db_projects.py	100.0% <100.0%> (ø)
...rage/src/simcore_service_storage/simcore_s3_dsm.py	94.0% <100.0%> (ø)
...r/src/simcore_service_webserver/storage_schemas.py	0.0% <0.0%> (-100.0%)	⬇️
...service_webserver/projects/_project_models_rest.py	0.0% <0.0%> (-100.0%)	⬇️
...es/web/server/src/simcore_service_webserver/log.py	0.0% <0.0%> (-94.8%)	⬇️
...erver/src/simcore_service_webserver/rest_models.py	0.0% <0.0%> (-92.7%)	⬇️
...simcore_service_webserver/director/director_api.py	0.0% <0.0%> (-76.0%)	⬇️
...rc/simcore_service_webserver/version_control_db.py	24.5% <0.0%> (-68.7%)	⬇️
...src/simcore_service_webserver/activity/handlers.py	29.3% <0.0%> (-63.8%)	⬇️
...ce_webserver/studies_dispatcher/_studies_access.py	22.7% <0.0%> (-63.7%)	⬇️
... and 656 more

[sanderegg]

pair-fixed. Thanks!

[codeclimate]

Code Climate has analyzed commit df05c01 and detected 0 issues on this pull request.

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication