Skip to content

Commit 936dba6

Browse files
committed
Upgrade to guzzlehttp/guzzle ~4.0
This version is known to fix certificate issues in some environments. Required changes: - New GuzzleHttp namespace - Catch different exception class - Use a Stream request body Closes #3.
1 parent ad27fad commit 936dba6

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

composer.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414
}
1515
],
1616
"require": {
17-
"php": ">=5.3",
18-
"guzzle/guzzle": ">=3.0",
17+
"php": ">=5.4",
18+
"guzzlehttp/guzzle": "~4.0",
1919
"satooshi/php-coveralls": "dev-master",
2020
"symfony/console": ">=2.0"
2121
},

src/CodeClimate/Bundle/TestReporterBundle/ApiClient.php

+7-3
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
<?php
22
namespace CodeClimate\Bundle\TestReporterBundle;
33

4-
use Guzzle\Http\Client;
5-
use Guzzle\Http\Exception\ClientErrorResponseException;
4+
use GuzzleHttp\Client;
5+
use GuzzleHttp\Stream;
6+
use GuzzleHttp\Exception\ClientException;
7+
use GuzzleHttp\Exception\ClientErrorResponseException;
68

79
class ApiClient
810
{
@@ -27,10 +29,12 @@ public function send($json)
2729

2830
$request->setHeader("User-Agent", "Code Climate (PHP Test Reporter v".Version::VERSION.")");
2931
$request->setHeader("Content-Type", "application/json");
30-
$request->setBody($json);
32+
$request->setBody(Stream\create($json));
3133

3234
try {
3335
$response = $this->client->send($request);
36+
} catch (ClientException $e) {
37+
$response = $e->getResponse();
3438
} catch (ClientErrorResponseException $e) {
3539
$response = $e->getResponse();
3640
}

0 commit comments

Comments
 (0)