adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2025-01-19 02:56:15 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix HttpSocket test to actually reset protocols for correct test. Modify HttpSocket so that Https with non-passed in protocol doesn't set it to tcp and fail test.

Browse source
This commit is contained in:
Melvin Ross 2014-06-12 09:37:03 -05:00
parent 460cc29fff
commit ad4dbdcee5
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/Cake/Network/Http/HttpSocket.php
View file

@ -341,7 +341,7 @@ class HttpSocket extends CakeSocket {
if (!empty($this->request['body']) && !isset($this->request['header']['Content-Length'])) {
$this->request['header']['Content-Length'] = strlen($this->request['body']);
}
if (isset($this->request['uri']['scheme']) && $this->request['uri']['scheme'] === 'https' && empty($this->config['protocol'])) {
if (isset($this->request['uri']['scheme']) && $this->request['uri']['scheme'] === 'https' && in_array($this->config['protocol'], array(false, 'tcp'))) {
$this->config['protocol'] = 'ssl';
}

5
lib/Cake/Test/Case/Network/Http/HttpSocketTest.php
View file

@ -493,7 +493,7 @@ class HttpSocketTest extends CakeTestCase {
)
)
),
'reset9' => array(
'reset10' => array(
'config.protocol' => 'ssl'
),
array(
@ -524,6 +524,9 @@ class HttpSocketTest extends CakeTestCase {
)
)
),
'reset11' => array(
'config.protocol' => 'ssl'
),
array(
'request' => array(
'method' => 'POST',