cloudinary / cloudinary_php

PHP extension for Cloudinary
https://cloudinary.com/documentation/php_integration
MIT License
389 stars 150 forks source link

Drop Guzzle requirement and replace with any PSR-18 HttpClient #396

Open tacman opened 11 months ago

tacman commented 11 months ago

The current version of this SDK requires Guzzle. My application uses Symfony's HttpClient (which allows for automatic retries and integration with the debug toolbar, compelling reasons for me to use it).

Rather than recommending this library switch, better would be use only require that any PSR-18 http client be installed, and use it.

Here's an article about how and why: https://medium.com/php-fig/psr-18-the-php-standard-for-http-clients-3254c58752bb

At a minimum, even if Guzzle is installed, the developer should be able to choose what client they want to use for the requests.

Thanks!