Revert "Merge pull request #97 from utopia-php/ser-728"

[hmacr]

This reverts commit be9999d, reversing changes made to e973371.

Summary by CodeRabbit

• Chores
  • Updated dependency version constraints to align compatibility requirements for HTTP fetch and validation libraries
  • Adjusted HTTP timeout settings for improved database connection reliability

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The pull request updates two dependency version constraints in composer.json: utopia-php/fetch from 0.5.* to ^0.4.2 and utopia-php/validators from 0.1.* to ^0.1.0. Additionally, the HTTP timeout configuration in src/Audit/Adapter/ClickHouse.php is modified from a 30,000 millisecond value to 30 seconds.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: reverting a previous merge commit. The title is clear, specific, and directly related to the changeset which reverts dependency version changes.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be9999d and 0be573c.

⛔ Files ignored due to path filters (1)

• composer.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

• composer.json
• src/Audit/Adapter/ClickHouse.php

🔇 Additional comments (2)

composer.json (1)

32-33: Verify API compatibility of downgraded dependencies.

This revert downgrades two dependencies:

• utopia-php/fetch from 0.5.* to ^0.4.2
• utopia-php/validators from 0.1.* to ^0.1.0

Both versions exist and are available on Packagist. No known security advisories were found. However, confirm these older versions are API-compatible with the current codebase usage in src/Audit/Adapter/ClickHouse.php (specifically the Utopia\Fetch\Client and Utopia\Validator\Hostname usage), particularly around any breaking changes in the setTimeout() semantics for fetch.

src/Audit/Adapter/ClickHouse.php (1)

75-75: The setTimeout(30) call is correct and represents a 30-second timeout, which is consistent with utopia-php framework conventions where all timeout values are expressed in seconds.

---

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.}