-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Call to undefined function DeepL\curl_version() in deeplcom\deepl-php\src\Translator.php:738 #29
Comments
Thanks for reporting this. I didn't want the spaghetti code in the |
Hi @Radon8472 , I tested my changes that remove this just now by building PHP from source without the cURL extension, but |
You are right. in My local dev system based on xampp, I dont have this error when I activate curl. 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 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. |
I tried to change the http-client like shown in the README section - Custom HTTP client.
What results in Error:
Here is my code to find this error:
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:
$libraryInfoStr
to " curl/$curlVer"The text was updated successfully, but these errors were encountered: