Repository navigation
Call to undefined function DeepL\curl_version() in deeplcom\deepl-php\src\Translator.php:738 #29
Description
I tried to change the http-client like shown in the README section - Custom HTTP client.
What results in Error:
Call to undefined function DeepL\curl_version() in deeplcom\deepl-php\src\Translator.php:738
Here is my code to find this error:
$client = new \GuzzleHttp\Client([
'handler' => new \GuzzleHttp\Handler\StreamHandler()
]);
$translator = new \DeepL\Translator($authKey);
$translator->translateText($text, $source, $target, [
TranslatorOptions::HTTP_CLIENT => $client
]);So it seems that even when you chance the HttpClient, the Translator Class still tries to use method/functions related to Curl HttpClient for building the user-agend string.
Suggestion to fix:
- check which Guzzle handler is used
- Only is a handler is used, that extends CurlHandler, set
$libraryInfoStrto " curl/$curlVer" - for other Handler, either don`t add this info, or find a way to get similar infos from the other Handler classes
Activity
JanEbbing commentedon Jun 30, 2023
Thanks for reporting this. I didn't want the spaghetti code in the
constructUserAgentString, but it seems it needs to know about a bunch of things to properly report. This will be fixed in the next patch.JanEbbing commentedon Jul 10, 2023
Hi @Radon8472 , I tested my changes that remove this just now by building PHP from source without the cURL extension, but
deepl-phphascurl-extas part of its requirements at the moment, so it won't install. Do you get this error with the cURL extension installed? You can check this for example withphp -i | grep curl. The output is empty when you don't have the extension installed.Radon8472 commentedon Jul 12, 2023
You are right. in My local dev system based on xampp, I dont have this error when I activate curl.
But when I push the files to my life system (where I don`t have curl) I get the error.
You are using guzzle, and guzzle can run fine without curl using the native php streams or socket based Handlers, so I would recommend to move the curl dependency from the
requiresection tosuggested.Otherwise you would stop everybody without curl extension from using deepl, and I see no reason for this, when you are using guzzle, what is made to be useable with native php too.