Какое условие поставить при недосутпности адреса в curl?

Question

[lasthero67]

Здравствуйте. Имеется WordPress 5.8.2. На сайте я использую библиотеку curl со следующей конструкцией

$ch = curl_init();
curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
curl_setopt($ch, CURLOPT_URL, $url . '?' . http_build_query($params));
$response = curl_exec($ch);
$data = new SimpleXMLElement($response);
print_r($response);
curl_close($ch);

где url - это адрес запроса, $params - параметры запроса. В последнее время страницы с curl периодически перестали открываться (WordPress пишет "На сайте возникла критическая ошибка."). Это происходит кратковременно, буквально при перезагрузке страницы все загружается. Предполагаю, что такая ошибка возникакт, когда адрес запроса недоступен. Как поставить условие, чтобы при недоступности адреса запроса дальнейшая обработка не происходила и страница оставалась доступной?

Comments

[Антон]

Предполагаю

Логи изучали?

[lasthero67]

Антон, нет. Как и где их можно посмотреть?

Answer 1

[rPman]

после вызова curl_exec сделай вызов curl_getinfo и проверь что именно тебе возвращает сервер, в частности CURLINFO_HTTP_CODE (или лучше CURLINFO_RESPONSE_CODE) ($info['http_code']) должен быть 200, если не так, то смотри
CURLINFO_SSL_VERIFYRESULT (список кодов смотри в комментариях) и CURLINFO_PROXY_SSL_VERIFYRESULT если используется прокси
смотри ответ CURLINFO_HTTP_CONNECTCODE, CURLINFO_OS_ERRNO
CURLINFO_CONDITION_UNMET

p.s. так же проверь $response, если он пустой то нет смысла пытаться его распарсить на xml, это тоже ошибка (это значит сервер вернул пустой ответ при это м не сгенерировав ошибки, такое бывает когда криворукие бакэнд программисты реализуют api)

Comments

[lasthero67]

Провеил $response. Возвращает 503 ошибку. Получается нужно спрашивать с тех, кто реализовал API, чтобы устраняли ошибку?

[rPman]

lasthero67, обычно этот код возвращают когда сервер не справляется с текущей нагрузкой, нормальный ответ владельца сервера - уменьшай частоту запросов

правильная реализация нагруженных веб сервисов подразумевает что в заголовках ответов будет что то типа request-limit-count и request-limit-timeout (не стандартизировано), из которых можно понять, сколько еще можно сделать запросов или как долго нужно ждать чтобы стало можно делать запрос, иначе клиентам придется постоянно переспрашивать через определенный таймаут сервис до получения ответа, что вредно как обоим сторонам.

[lasthero67]

rPman, понял. А как мне в этом случае остановить дальнейшее выполнение кода, чтобы не было критической ошибки на странице?

[rPman]

вопрос не понял, что значит не было критической ошибки? а твоя логика может работать без ответа от этого api сервера?

я же все описал.
по уму в данной ситуации нужно подождать (буквально sleep) какое то время и сделать повторную попытку к серверу, и так в цикле, пока не истечет твой таймаут, который ты готов отвести на это

совсем правильно, отдать ошибку, но клиент должен уметь ее обрабатывать и делать повторный запрос

[lasthero67]

сделать повторную попытку к серверу, и так в цикле, пока не истечет твой таймаут, который ты готов отвести на это

Вот это как раз интересует. Как это реализовать? Тут мне по сути даже таймаут не нужен, а просто до получения ответа, т.к. там недоступность очень кратковременная.

[rPman]

таймауты короче секунды делают с помощью usleep, он в микросекундах принимает

без таймаутов будет натуральный ddos

совет, делай плавающий sleep, постепенно его увеличивая, например в 2 раза до некоторого порога потом постоянный

[lasthero67]

rPman, какое условие для $response будет?

Answer 2

[spaceatmoon]

Используй всегда ограничение на время ожидания ответа.

curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, 5); //5 секунд

Comments

[lasthero67]

Не помогает

[spaceatmoon]

lasthero67,

$response = curl_exec($ch);
$data = new SimpleXMLElement($response);

Это херовый кусок кода. Добавь что-то типа этого. Парси только если ошибок нет.

$curl_errno = curl_errno($ch);
 $curl_error = curl_error($ch);
 curl_close($ch);
 if ($curl_errno > 0) {
        echo "cURL Error ($curl_errno): $curl_error\n";
 } else {
        echo "Data received: $data\n";
 }

[lasthero67]

spaceatmoon, такая конструкция ошибку 503 не отлавливает

[lasthero67]

spaceatmoon, сделал так

curl_setopt($ch, CURLOPT_FAILONERROR, true);
$response = curl_exec($ch);
if (curl_errno($ch) > 0) {

} else {
	$data = new SimpleXMLElement($response);
}
curl_close($ch);

вроде получилось