Skip to content
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

Fix ssl error #5

Merged
merged 1 commit into from
Aug 25, 2016
Merged

Fix ssl error #5

merged 1 commit into from
Aug 25, 2016

Conversation

exodusanto
Copy link

@exodusanto exodusanto commented Aug 22, 2016

Update paypal sdk to 1.6.4

  • CURLOPT_SSLVERSION fix

@ash-rain ash-rain merged commit 7eeeb1a into net-shell:master Aug 25, 2016
@ash-rain
Copy link

Thanks, can you add more details about the error thrown?

@exodusanto
Copy link
Author

exodusanto commented Aug 25, 2016

Yes of course, in some cases there was a problem in the request to api.sandbox.paypal.com, to fix it you have to change this option $defaultCurlOptions[CURLOPT_SSLVERSION] in PayPalHttpConfig.

With 1.6.4 Paypal fix this problem #474

I tested also 1.7.0 in sandbox and it works, it's your choice if change it to stable version.

Thank you :)

@ash-rain
Copy link

I've been testing. Seems to be fine with this version so I'll just leave it and monitor for a few weeks.
Thanks for the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants