Fix PHP 8.5 curl_close() deprecation#48
Conversation
Refs: CLO-4291
Greptile SummaryRemoves the no-op Confidence Score: 5/5Safe to merge — minimal, correct removal of deprecated no-op calls with no behavioral impact. The change is a straightforward removal of a function that has been a documented no-op since PHP 8.0. No logic is altered, no resource handling changes, and the fix is applied consistently across all four affected adapters. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "Remove deprecated curl_close() calls (PH..." | Re-trigger Greptile |
|
Superseded by full migration to |
Summary
\curl_close($ch)calls in all four logger adapters (Sentry, AppSignal, LogOwl, Raygun) that triggerDeprecatednotices on PHP 8.5.Why
curl_close()has been a no-op since PHP 8.0 and is now formally deprecated in 8.5. The curl handle is freed when the variable goes out of scope.Test plan
composer lint(Pint) — passcomposer check(PHPStan max) — pass on PHP 8.3