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

PHP 5.6 Compatibility Issue #48

Open
KAB8609 opened this issue Jul 28, 2015 · 2 comments
Open

PHP 5.6 Compatibility Issue #48

KAB8609 opened this issue Jul 28, 2015 · 2 comments

Comments

@KAB8609
Copy link

KAB8609 commented Jul 28, 2015

The WSDL is not correctly sending off in a PHP 5.6 environment. This is a bug that we have found on several WSDL calls in Magento.

The error in the PHP logs is:
_PHP Error [select all]
SoapClient::_doRequest(): SSL operation failed with code 1. OpenSSL Error messages: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

The fix is
app/code/community/OnePica/AvaTax/lib/classes/TaxServiceSoap.class.php
Starting on line 69

$this->client = new DynamicSoapClient (
$config->taxWSDL,
array
(
'location' => $config->url.$config->taxService,
'trace' => $config->trace,
'classmap' => TaxServiceSoap::$classmap,
'stream_context' => stream_context_create(array('ssl' => array('verify_peer' => false,'verify_peer_name' => false)))
),
$config
);

rredko-sysiq pushed a commit that referenced this issue Aug 20, 2015
- Remove concatenation in require_once statements
@rredko-sysiq
Copy link
Contributor

Thank you for information. We will investigate this issue. We are using Avalara AvaTax SDK so better to ask them to update this.

@ihor-sviziev
Copy link

@KAB8609 we're using this module on php 5.6 without any issues. Change that you proposed can't be applied by security reasons. We must verify ssl certificate for for this domain, coz it must be valid.
If you have such issues - please check that certificate for that domain isn't replaced to another one.

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

No branches or pull requests

3 participants