🐛♻️ webserver error handling: Fix incorrect error logging for web.HTTPCreated; cleanup success response handling

[pcrespov]

What do these changes do?

• 🐛 Fixed incorrect error logging when web.HTTPCreated was raised in LRT

log_level=ERROR | log_timestamp=2025-07-05 09:25:20,343 | log_source=servicelib.long_running_tasks.http_endpoint_responses:get_task_result(45) | log_uid=None | log_oec=OEC:2f0d6d1c07ab-1751707520343| log_msg=task_id='POST%20%2Fv0%2Fprojects%3Ffrom_study%3D3946db48-5646-11f0-a416-0242ac17e851.865277dc-e461-41e0-b0b7-37f111f836ad' raised an exception.
{
  "exception_type": "<class 'aiohttp.web_exceptions.HTTPCreated'>",
  "exception_string": "Created",
  "exception_causes": "",
  "error_code": "OEC:2f0d6d1c07ab-1751707520343",
  "context": {
    "task_context": {
      "servicelib.request_keys.userid": 371,
      "simcore_service_webserver.constants.RQ_PRODUCT_KEY": "s4lacad"
    },
    "task_id": "POST%20%2Fv0%2Fprojects%3Ffrom_study%3D3946db48-5646-11f0-a416-0242ac17e851.865277dc-e461-41e0-b0b7-37f111f836ad"
  },
  "tip": null
}
Traceback (most recent call last):
  File "/home/scu/.venv/lib/python3.11/site-packages/servicelib/long_running_tasks/http_endpoint_responses.py", line 35, in get_task_result
    task_result = tasks_manager.get_task_result(
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/scu/.venv/lib/python3.11/site-packages/servicelib/long_running_tasks/task.py", line 254, in get_task_result
    return tracked_task.task.result()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/scu/.venv/lib/python3.11/site-packages/servicelib/long_running_tasks/task.py", line 395, in _progress_task
    return await handler(progress, **task_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/scu/.venv/lib/python3.11/site-packages/simcore_service_webserver/projects/_crud_api_create.py", line 484, in create_project
    raise web.HTTPCreated(
aiohttp.web_exceptions.HTTPCreated: Created

• Refactored create_project to return web.HTTPCreated instead of raising it

• 

• ♻️ Cleaned up web.HTTPSuccessful handling in rest_middleware

Related issue/s

• Follows up from 🐛 web-api: Fixes handling of unexpected errors #7939

How to test

• packages/service-library/tests/aiohttp/test_rest_middlewares.py
• services/web/server/tests/unit/isolated/test_exception_handling_factory.py

Dev-ops

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 87.62%. Comparing base (904b669) to head (22c808c).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7952      +/-   ##
==========================================
- Coverage   87.82%   87.62%   -0.21%     
==========================================
  Files        1848     1422     -426     
  Lines       71250    59060   -12190     
  Branches     1250      626     -624     
==========================================
- Hits        62575    51750   -10825     
+ Misses       8313     7100    -1213     
+ Partials      362      210     -152     

Flag	Coverage Δ
integrationtests	63.23% <50.00%> (-1.04%)	⬇️
unittests	86.04% <75.00%> (-0.38%)	⬇️

Components	Coverage Δ
api	∅ <ø> (∅)
pkg_aws_library	∅ <ø> (∅)
pkg_celery_library	∅ <ø> (∅)
pkg_dask_task_models_library	∅ <ø> (∅)
pkg_models_library	∅ <ø> (∅)
pkg_notifications_library	∅ <ø> (∅)
pkg_postgres_database	∅ <ø> (∅)
pkg_service_integration	∅ <ø> (∅)
pkg_service_library	71.54% <70.00%> (-0.02%)	⬇️
pkg_settings_library	∅ <ø> (∅)
pkg_simcore_sdk	85.05% <ø> (+0.05%)	⬆️
agent	96.29% <ø> (ø)
api_server	92.31% <ø> (ø)
autoscaling	96.03% <ø> (ø)
catalog	92.29% <ø> (ø)
clusters_keeper	99.13% <ø> (ø)
dask_sidecar	91.79% <ø> (ø)
datcore_adapter	97.94% <ø> (ø)
director	76.73% <ø> (ø)
director_v2	91.12% <ø> (+0.02%)	⬆️
dynamic_scheduler	96.69% <ø> (ø)
dynamic_sidecar	90.09% <ø> (ø)
efs_guardian	89.65% <ø> (ø)
invitations	93.60% <ø> (ø)
payments	92.57% <ø> (ø)
resource_usage_tracker	89.00% <ø> (-0.11%)	⬇️
storage	86.31% <ø> (ø)
webclient	∅ <ø> (∅)
webserver	87.31% <100.00%> (-0.33%)	⬇️

---

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 904b669...22c808c. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pcrespov]

@mergify queue

[mergify]

queue

🟠 Waiting for conditions to match

• -closed [📌 queue requirement]
• -conflict [📌 queue requirement]
• -draft [📌 queue requirement]
• any of: [📌 queue -> configuration change requirements]
  • -mergify-configuration-changed
  • check-success = Configuration changed
• any of: [🔀 queue conditions]
  • all of: [📌 queue conditions of queue default]
    • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
    • #approved-reviews-by>=2
    • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
    • #changes-requested-reviews-by=0
    • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
    • #review-threads-unresolved=0
    • -conflict
    • -draft
    • base=master
    • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
    • label!=🤖-do-not-merge
    • label=🤖-automerge
    • any of: [🛡 GitHub branch protection]
      • check-skipped = deploy to dockerhub
      • check-neutral = deploy to dockerhub
      • check-success = deploy to dockerhub
    • any of: [🛡 GitHub branch protection]
      • check-success = system-tests
      • check-neutral = system-tests
      • check-skipped = system-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = unit-tests
      • check-neutral = unit-tests
      • check-skipped = unit-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = check OAS' are up to date
      • check-neutral = check OAS' are up to date
      • check-skipped = check OAS' are up to date
    • any of: [🛡 GitHub branch protection]
      • check-success = integration-tests
      • check-neutral = integration-tests
      • check-skipped = integration-tests
    • any of: [🛡 GitHub branch protection]
      • check-success = build-test-images (frontend) / build-test-images
      • check-neutral = build-test-images (frontend) / build-test-images
      • check-skipped = build-test-images (frontend) / build-test-images

[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