Skip to content

Commit e8389ac

Browse files
committed
Catch exceptions
1 parent 368d35a commit e8389ac

File tree

1 file changed

+13
-14
lines changed

1 file changed

+13
-14
lines changed

Diff for: src/Lodestone/Http/Http.php

+13-14
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
use Lodestone\Parser\Parser;
1010
use Ramsey\Uuid\Uuid;
1111
use Symfony\Component\HttpClient\CurlHttpClient;
12+
use Symfony\Component\HttpClient\Exception\ClientException;
1213

1314
class Http
1415
{
@@ -111,24 +112,22 @@ public function settle()
111112

112113
// grab request id
113114
$requestId = $userdata['request_id'];
114-
115-
// if it wasn't a 200, return error
116-
if ($response->getStatusCode() != 200) {
117-
$content[$requestId] = (Object)[
118-
'Error' => true,
119-
'StatusCode' => $response->getStatusCode()
120-
];
121-
continue;
122-
}
123-
115+
124116
// grab the parser class name
125117
/** @var Parser $parser */
126118
$parser = new $userdata['parser']($userdata);
127119

128-
// handle response
129-
$content[$requestId] = $parser->handle(
130-
$response->getContent()
131-
);
120+
try {
121+
// handle response
122+
$content[$requestId] = $parser->handle(
123+
$response->getContent()
124+
);
125+
} catch (ClientException $ex) {
126+
$content[$requestId] = (Object)[
127+
'Error' => true,
128+
'StatusCode' => $ex->getCode()
129+
];
130+
}
132131
}
133132
}
134133

0 commit comments

Comments
 (0)