Migrate adapters to utopia-php/fetch#49
Conversation
Greptile SummaryThis PR replaces raw Confidence Score: 5/5This PR is safe to merge — it is a straightforward internal HTTP-client substitution with no public API changes and no logic regressions. No P0 or P1 findings. Timeout unit conversion, body encoding, exception handling, and method names all match the fetch library's v1.1.2 contract. The previously flagged Raygun doc-comment issue is resolved in this diff. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "Fix Raygun docblock copy-paste reference" | Re-trigger Greptile |
The property comment incorrectly referenced AppSignal's setup path. Updated to reference Raygun's correct API-key location.
Summary
utopia-php/fetch(^1.1) in all four adapters: Sentry, AppSignal, LogOwl, Raygun.fetchis purely an internal substitution.phpconstraint to>=8.1(fetch's minimum). Suggest a major-version tag for downstream.Why
curl_close()deprecation as a side effect (no more rawcurl_*calls).Test plan
composer lint(Pint)composer check(PHPStan max)composer test(unit + e2e)