fix: preserve HTTPException status in update_docling_preset (#1586)

[hunterxtang]

Fixes #1586

Summary by CodeRabbit

• Bug Fixes
  • Fixed error handling for the docling-preset endpoint so client-side errors (e.g., invalid preset requests) now return their original HTTP status codes and messages instead of being converted to a generic server error. This restores correct status propagation to clients and prevents misleading 500 responses for client-induced failures.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 32a96009-e468-4d56-8b45-6fd8de20c723

📥 Commits

Reviewing files that changed from the base of the PR and between e3de2b3 and cd3713f.

📒 Files selected for processing (1)

• src/api/settings/endpoints.py

🚧 Files skipped from review as they are similar to previous changes (1)

• src/api/settings/endpoints.py

---

Walkthrough

The deprecated /settings/docling-preset endpoint now re-raises caught HTTPException instances in update_docling_preset, preventing them from being masked as generic 500 errors and preserving original HTTP status codes and messages.

Changes

HTTP Exception Handling Fix

Layer / File(s)	Summary
HTTP exception preservation in docling-preset endpoint src/api/settings/endpoints.py	The update_docling_preset function adds an except HTTPException block that re-raises exceptions to preserve intended HTTP status codes instead of overriding them with a generic 500 response.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

bug

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: fixing HTTPException status preservation in the update_docling_preset endpoint, with an issue reference.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}