Compatibility issues with PHP 8.5 #1127

Activity

1. alekr-jsausa commented on Feb 3, 2026

   

   alekr-jsausa

   on Feb 3, 2026

   More actions

   I am running into this issue as well. I think it can be fixed by replacing the offending line with:

   if (PHP_MAJOR_VERSION < 8) {
       curl_close($channel);
   }

   to keep compatibility with PHP 7. I'm happy to file a PR for this if that's what is needed to get this fixed.

   👍React with 👍1Reacted by Edward Slipszenko
2. slipszenko commented on Feb 6, 2026

   

   slipszenko

   on Feb 6, 2026

   Last edited by slipszenko

   More actions

   This is in a library used by this library, and has a couple of PRs open:

   • chore: Remove unnecessary curl_close call php-http-client#175
   • chore: fix PHP 8.5 curl_close() deprecation warning php-http-client#174

   It'd be great to see one of those merged.
3. andreas-aeschlimann commented on Feb 8, 2026

   

   andreas-aeschlimann

   on Feb 8, 2026

   Contributor

   More actions

   If you do not want to downgrade PHP or use an own fork of SendGrid, you can use a drop-in hack into their namespace as follows:

   Create a PHP file Curl.php somewhere:

   <?php
   
   namespace SendGrid;
   
   function curl_close($ch): void {
       if (is_resource($ch)) {
           unset($ch);
       }
   }
   

   Import this file BEFORE the autoload/ClassLoader system, for example in your custom autoload.php:

   <?php
   
   use Composer\Autoload\ClassLoader;
   
   require __DIR__ . '/Compat/SendGrid/Curl.php'; // TODO: temporary fix
   
   require_once __DIR__ . '/../vendor/autoload.php';
   
   $classLoader = new ClassLoader();
   $classLoader->addPsr4("YourProject\\", __DIR__, true);
   $classLoader->register();
   

   This works because SendGrid is not using \curl_close, but curl_close in their files.
4. esetnik commented on Jun 18, 2026

   

   esetnik

   on Jun 18, 2026

   More actions

   sendgrid/php-http-client#176 was merged

   👍React with 👍1Reacted by Edward Slipszenko