L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-38509 Unify curl class usage in the add-on installer tool

Browse Source 
Both classes using cURL features now access it via the core curl wrapper
class. Credit goes to Dan Poltawski for spotting the previous discrepancy
during the integration review.
This commit is contained in:
David Mudrák
2013-04-02 23:11:01 +02:00
parent c9c4d0b24e
commit 3e6a8aeb7c
2 changed files with 44 additions and 68 deletions
Download Patch File Download Diff File Expand all files Collapse all files
admin/tool/installaddon/classes/pluginfo_client.php
+15 -8
View File
@@ -80,21 +80,28 @@ class tool_installaddon_pluginfo_client {
require_once($CFG->libdir.'/filelib.php');
$curl = new curl(array('proxy' => true));
$response = $curl->get(
$this->service_request_url(),
$this->service_request_params($component, $version),
$this->service_request_options());
$curlerrno = $curl->get_errno();
if (!empty($curlerrno)) {
throw new tool_installaddon_pluginfo_exception('err_response_curl', 'cURL error '.$curlerrno.': '.$curl->error);
}
$curlinfo = $curl->get_info();
/* if ($curlinfo['http_code'] == 404) {
throw new tool_installaddon_pluginfo_exception('err_response_http_code_404');
}*/
if ($curlinfo['http_code'] != 200) {
throw new tool_installaddon_pluginfo_exception('err_response_http_code', $curlinfo['http_code']);
if (!empty($curlerrno)) {
throw new tool_installaddon_pluginfo_exception('err_curl_exec', array(
'url' => $curlinfo['url'], 'errno' => $curlerrno, 'error' => $curl->error));
} else if ($curlinfo['http_code'] != 200) {
throw new tool_installaddon_pluginfo_exception('err_curl_http_code', array(
'url' => $curlinfo['url'], 'http_code' => $curlinfo['http_code']));
} else if (isset($curlinfo['ssl_verify_result']) and $curlinfo['ssl_verify_result'] != 0) {
throw new tool_installaddon_pluginfo_exception('err_curl_ssl_verify', array(
'url' => $curlinfo['url'], 'ssl_verify_result' => $curlinfo['ssl_verify_result']));
}
return $response;
}