Chore: Upgrade HTTP

[Meldiron]

What does this PR do?

• fixes double slash 5xx errors
• improves domain validator

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

• (Related PR or issue)

Checklist

• Have you read the Contributing Guidelines on issues?
• If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

[coderabbitai]

📝 Walkthrough

Walkthrough

• tests/e2e/Services/Proxy/ProxyCustomServerTest.php: Updated testCreateAPIRule() to expect 400 responses (instead of 201) for APIRule creations with wss://domain and domain/some-path; removed corresponding cleanup steps and comments. No other test logic changed.
• app/controllers/api/health.php: Modified certificate-domain validation to allow either URL or Domain via Multiple([AnyOf([URL, Domain]), PublicDomain()]); added imports for AnyOf and URL. No other endpoints or logic changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

• eldadfux

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.

---

📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 6f5d038 and 52af439.

📒 Files selected for processing (1)

• app/controllers/api/health.php (2 hunks)

🧰 Additional context used

🧬 Code Graph Analysis (1)

app/controllers/api/health.php (2)

src/Appwrite/URL/URL.php (1)

• URL (5-121)

src/Appwrite/GraphQL/Types/Mapper.php (1)

• param (253-352)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: Setup & Build Appwrite Image
• GitHub Check: Setup & Build Appwrite Image
• GitHub Check: Linter
• GitHub Check: scan

🔇 Additional comments (3)

app/controllers/api/health.php (3)

26-26: LGTM: Needed import for composite validation

Importing AnyOf is appropriate for the new validator composition.

---

31-31: LGTM: URL validator import

Importing Utopia\Validator\URL keeps the param validator concise and consistent.

---

402-402: Confirm PublicDomain behavior and ->param signature

I wasn’t able to locate a local PublicDomain implementation (it’s likely coming from a vendor package) nor verify the exact ->param(…, Multiple::TYPE_STRING, …) signature usage. Please manually confirm:

• PublicDomain validation
  Does it accept full URLs or only raw hostnames? If it rejects URLs, defer PublicDomain until after you extract the host (e.g., in your action callback).
• ->param signature consistency
  Review other controllers to ensure the trailing boolean “required” flag is placed correctly when using Multiple::TYPE_STRING.

Once confirmed, you can tighten the validator and update the description. For example (pending your verification):

-    ->param('domain', null, new Multiple([new AnyOf([new URL(), new Domain()]), new PublicDomain()]), Multiple::TYPE_STRING, 'Domain name')
+    ->param('domain', null, new Multiple([new AnyOf([new URL(), new Domain()])]), Multiple::TYPE_STRING, 'Domain name or URL')

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore-upgrade-http

---

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[github-actions]

Security Scan Results for PR

Docker Image Scan Results

Package	Version	Vulnerability	Severity
binutils	2.42-r0	CVE-2025-0840	HIGH
git	2.45.3-r0	CVE-2025-46334	HIGH
git	2.45.3-r0	CVE-2025-48384	HIGH
git	2.45.3-r0	CVE-2025-48385	HIGH
git-init-template	2.45.3-r0	CVE-2025-46334	HIGH
git-init-template	2.45.3-r0	CVE-2025-48384	HIGH
git-init-template	2.45.3-r0	CVE-2025-48385	HIGH
icu	74.2-r0	CVE-2025-5222	HIGH
icu-data-en	74.2-r0	CVE-2025-5222	HIGH
icu-dev	74.2-r0	CVE-2025-5222	HIGH
icu-libs	74.2-r0	CVE-2025-5222	HIGH
libexpat	2.6.4-r0	CVE-2024-8176	HIGH
libxml2	2.12.7-r0	CVE-2024-56171	HIGH
libxml2	2.12.7-r0	CVE-2025-24928	HIGH
libxml2	2.12.7-r0	CVE-2025-27113	HIGH
libxml2	2.12.7-r0	CVE-2025-32414	HIGH
libxml2	2.12.7-r0	CVE-2025-32415	HIGH
pyc	3.12.9-r0	CVE-2024-12718	HIGH
pyc	3.12.9-r0	CVE-2025-4138	HIGH
pyc	3.12.9-r0	CVE-2025-4330	HIGH
pyc	3.12.9-r0	CVE-2025-4517	HIGH
python3	3.12.9-r0	CVE-2024-12718	HIGH
python3	3.12.9-r0	CVE-2025-4138	HIGH
python3	3.12.9-r0	CVE-2025-4330	HIGH
python3	3.12.9-r0	CVE-2025-4517	HIGH
python3-pyc	3.12.9-r0	CVE-2024-12718	HIGH
python3-pyc	3.12.9-r0	CVE-2025-4138	HIGH
python3-pyc	3.12.9-r0	CVE-2025-4330	HIGH
python3-pyc	3.12.9-r0	CVE-2025-4517	HIGH
python3-pycache-pyc0	3.12.9-r0	CVE-2024-12718	HIGH
python3-pycache-pyc0	3.12.9-r0	CVE-2025-4138	HIGH
python3-pycache-pyc0	3.12.9-r0	CVE-2025-4330	HIGH
python3-pycache-pyc0	3.12.9-r0	CVE-2025-4517	HIGH
sqlite-libs	3.45.3-r1	CVE-2025-29087	HIGH
xz	5.6.2-r0	CVE-2025-31115	HIGH
xz-libs	5.6.2-r0	CVE-2025-31115	HIGH
golang.org/x/crypto	v0.31.0	CVE-2025-22869	HIGH
golang.org/x/oauth2	v0.24.0	CVE-2025-22868	HIGH
stdlib	1.22.10	CVE-2025-47907	HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

[github-actions]

✨ Benchmark results

• Requests per second: 981
• Requests with 200 status code: 176,581
• P99 latency: 0.18710324

⚡ Benchmark Comparison

Metric	This PR	Latest version
RPS	981	984
200	176,581	177,201
P99	0.18710324	0.196449462