🐛 catalog-api: Fixes validation errors on /services entrypoint

[pcrespov]

What do these changes do?

This PR fixes a validation issue caused by invalid values in the database.

A request to the catalog service was failing due to an invalid value in the database:

curl http://master_catalog:8000/v0/services/simcore%2Fservices%2Fdynamic%2Fjupyter-math/3.0.3?user_id=530
{"errors":["1 validation errors:\n  {'type': 'url_parsing', 'loc': ('response', 'icon'), 'msg': 'Input should be a valid URL, input is empty', 'input': '', 'ctx': {'error': 'input is empty'}}\n"]}

The issue occurred because the icon column in the second row was set to an empty string instead of either a valid URL or NULL.

This could have been introduced manually in the database or as a result of an unintended transformation between models in the system.

To address this, we implemented the following changes:

• Database Migration: Replaces existing empty strings with NULL in nullable string columns.
• Validation Mechanism: Prevents writing empty strings to the database at the model level.

Since these values should not be modified in the database directly, we consider additional constraints at the database level unnecessary, as the existing code-level checks are sufficient.

Related issue/s

• Issue reported by @GitHK in master deploy

How to test

Dev-ops

None

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.22%. Comparing base (29816b1) to head (52fc53f).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7268      +/-   ##
==========================================
+ Coverage   86.97%   88.22%   +1.24%     
==========================================
  Files        1686     1454     -232     
  Lines       65340    57900    -7440     
  Branches     1115      616     -499     
==========================================
- Hits        56832    51084    -5748     
+ Misses       8190     6644    -1546     
+ Partials      318      172     -146     

Flag	Coverage Δ
integrationtests	65.47% <ø> (+0.04%)	⬆️
unittests	87.15% <100.00%> (+1.15%)	⬆️

Components	Coverage Δ
api	∅ <ø> (∅)
pkg_aws_library	∅ <ø> (∅)
pkg_dask_task_models_library	∅ <ø> (∅)
pkg_models_library	91.56% <100.00%> (ø)
pkg_notifications_library	84.57% <ø> (ø)
pkg_postgres_database	88.28% <ø> (ø)
pkg_service_integration	70.03% <ø> (ø)
pkg_service_library	∅ <ø> (∅)
pkg_settings_library	∅ <ø> (∅)
pkg_simcore_sdk	85.08% <ø> (ø)
agent	96.46% <ø> (ø)
api_server	90.56% <ø> (ø)
autoscaling	96.08% <ø> (ø)
catalog	91.73% <100.00%> (+0.01%)	⬆️
clusters_keeper	99.24% <ø> (ø)
dask_sidecar	91.25% <ø> (ø)
datcore_adapter	93.19% <ø> (ø)
director	76.59% <ø> (-0.19%)	⬇️
director_v2	91.30% <ø> (ø)
dynamic_scheduler	97.33% <ø> (ø)
dynamic_sidecar	89.74% <ø> (ø)
efs_guardian	90.25% <ø> (ø)
invitations	93.28% <ø> (ø)
osparc_gateway_server	∅ <ø> (∅)
payments	92.66% <ø> (ø)
resource_usage_tracker	89.08% <ø> (-0.06%)	⬇️
storage	86.65% <ø> (-0.06%)	⬇️
webclient	∅ <ø> (∅)
webserver	84.68% <ø> (ø)

---

Continue to review full report in Codecov by Sentry.

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

[GitHK]

🙏 thanks a lot!

[odeimaiz]

Does this mean that an empty string is not a valid value?

[pcrespov]

Does this mean that an empty string is not a valid value?

@odeimaiz in the database is not an expected value when the column is nullable. At least for the columns we are speaking about in this PR

• the front-end should not be affected by this constraint because there is a mechanism in place for these cases that will replace empty strings by null values.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud