-
Notifications
You must be signed in to change notification settings - Fork 176
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
TLS 1.2 support #103
Comments
Yes, I have been getting errors on our test server for about a week, though oddly not from my local machine. The test server is not my personal paypal account, though. I need this to continue testing my application, though. D: I can try using my personal sandbox credentials, but I fear that account is soon to be affected as well. Does anyone know of a temporary fix? |
You can pass a customized Guzzle\Http\Client with CURLOPT_SSLVERSION => 6 El mar., 26 de ene. de 2016 19:02, Darryl Meganoski <
|
What you will probably need to do as well, and what we ended up having to do, is to upgrade your version of libcurl and related libraries to support TLS 1.2. The version of TLS is actually auto-negotiated by the client (curl) and the server (apache/nginx/whatever) so if your libcurl doesn't support TLS 1.2 you're going nowhere. If you're on CentOS 6 for example, the supplied libcurl does not support TLS 1.2. Try this:
|
My understanding is that libcurl versions > 7.40 should do the trick. libcurl for CentOS 6 is 7.19. |
My local machine has 7.39, I'll have to check the server. Thanks for the tips, I'll see if I can get this working later today. |
Scratch that, I'm having issues in my live environment with that version. But not in Vagrant. Odd. |
I've updated ran a composer update, confirmed latest version of omnipay/paypal as well as performed the libcurl update instructions above: Package libcurl-7.47.0-1.0.cf.rhel6.x86_64 installed and latest version Still getting the "[curl] 35" error when using Sandbox. No mention of the SSLv3 error however as in #104 Any Ideas? Also, the rpm address you mention http://www.city-fan.org/ftp/contrib/yum-repo/city-fan.org-release-1-12.rhel6.noarch.rpm has since been moved and now causes a 404.. I grabbed: http://www.city-fan.org/ftp/contrib/yum-repo/city-fan.org-release-1-13.rhel6.noarch.rpm --- Update --- I also performed the command line test: and got the success message "PayPal_Connection_OK bool(true)" , so I'm pretty sure the server is ok per: https://github.com/paypal/TLS-update --- Final Update --- As a last effort, did an httpd restart and everything seems to be working so far...sorry for the confusion. I can delete this or leave it in case it might help someone else. |
Ah OK -- yes a http restart would be required to pick up the newer version of libcurl, in fact I would be inclined to do a reboot when updating such a low level library as it could be embedded in all sorts of stuff. |
Since this was top result in my search, I will leave this here for MAMP users with this issue: paypal/PayPal-PHP-SDK#484 (comment) |
According to https://devblog.paypal.com/upcoming-security-changes-notice/ Paypal is turning off SSL support in June 17, but I'm receiving errors from apps on sandbox right now as well.
Should we expect the library to change to TLS 1.2 by default?
Is there anything I can do to help you with that?
Cheers!
The text was updated successfully, but these errors were encountered: