Почему фид не открывается как xml?

Question

[Ilia Malashko]

Здравствуйте.

Фиды с товарами генерируем через стандартный функционал 1с-Битрикса.

Фиды обновляются каждые 30 мин, чаще всего с ними все хорошо, но иногда php интерпритирует его как текст и при открытии файла клиенты сталкиваются с ошибками.
Фид пожно получить по этой ссылке. Сейчас проблема фиксируется.

Я сохранил версии когда файл открывается как xml и когда как текст. Я не нашел отличий в этих файлах.

На сервере фид лежит в формате php, но отдается как xml:

$disableReferers = false;
if (!isset($_GET["referer1"]) || strlen($_GET["referer1"])<=0) $_GET["referer1"] = "yandext";
$strReferer1 = htmlspecialchars($_GET["referer1"]);
if (!isset($_GET["referer2"]) || strlen($_GET["referer2"]) <= 0) $_GET["referer2"] = "";
$strReferer2 = htmlspecialchars($_GET["referer2"]);
header("Content-Type: text/xml; charset=windows-1251");
echo "<"."?xml version=\"1.0\" encoding=\"windows-1251\"?".">"?>
<!DOCTYPE yml_catalog SYSTEM "shops.dtd">
<yml_catalog date="2024-11-29 09:06">
<shop>
<name>wisell.ru</name>
<company>wisell.ru</company>
<url>https://wisell.ru</url>
<platform>1C-Bitrix</platform>
<currencies>
<currency id="RUB" rate="1" />
// ... ...

Помогите разобраться в чем проблема. Спасибо.

Comments

[alexalexes]

Я сохранил версии когда файл открывается как xml и когда как текст.

Спарсите эти файлы обратно в DOM объект и посмотрите на ошибки парсера.

[Ипатьев]

Поясните, что вы имеете в виду, говоря "php интерпретирует его как текст".
При чём тут РНР? Что значит "интерпретирует как текст"?

[Gip]

что-то мне говорит о том, что ошибка у вас где-то в другом месте. оба файла - валидный XML. Не поленился проверил - оба нормально загружаются в пхп через XMLReader::open.

XMLReader::open('https://wisell.ru/bitrix/catalog_export/megamarket.php');

тоже работает нормально и всё прекрасно загружает.

Что значит "как текст" как вы это определили?

[Ilia Malashko]

Ипатьев, php тут в целом не при чем. XML сохранен в файле *.php, что бы это никого не смутило я решил пояснить что хоть файл и php в нем задается заголовок с типом text/xml и он должен открываться как xml

[Gip]

Ilia Malashko, так он и открывается. хоть браузером открывай. другое дело что он у вас отдается с помощью echo, а файл не маленький и если будет плохой коннект у открывающего то могут быть нюансы. в чём надобность генерить такой пхп файл если можно сохранить обычный статичный xml? у вас веб-сервер просто может не отдать в какой-то момент все эти 13 мегабайт через echo и получите невалидный xml 'как текст'. а если кто-то запустит в потоков 50 гет запросы на этот урл что будет догадываетесь?)

[Ипатьев]

Ilia Malashko, вы вообще помните, что в своем вопросе писали? При чем здесь "сохранен в файле *.php"?
У вас написано "php интерпретирует его как текст".
ЧТО ЭТО ЗНАЧИТ вы можете русским языком пояснить? КАКОЙ РНР? Что значит "интерпретирует"? Откуда вы вообще взяли это "как текст"?

[Ilia Malashko]

Gip, фиды собираются через интерфейс 1с-Битрикс, и это стандартный механизм генерации готовых фидов с данными. Меня давно это раздражает, я не понимаю зачем они так реализовали, приходится дергать apache хотя можно было просто отдавать данные статикой через nginx.
Переписывать все выгрузки нет ресурсов сейчас, попробую увеличить объем памяти для обработки php

[Ilia Malashko]

Ипатьев, текст с капслоком и без усваивается одинаково, не утруждайте себя переключением.
Обычно при получении данных с файла с помощью curl я могу работать с полученными данными как с xml, например

$sXML = download_page($url);
$xml = simplexml_load_string($sXML);

но сейчас я так же получаю информацию из этого файла но в $xml будет пусто.

[Gip]

Ilia Malashko, я бы всё равно какой-нибудь примитивный скрипт написал типа

file_put_contents('feed.xml', file_get_contents('https://wisell.ru/bitrix/catalog_export/megamarket.php'));

дёргал бы его кроном раз в полчаса, и отдавал бы клиентам статику, которую он сохраняет, а этот урл спрятал бы и закрыл для внешки вообще. если уж битрикс такая бяка

[Ilia Malashko]

Gip, я думал об этом. Но очень похоже на костыль, хотел разобраться почему так. Напишу еще в ТП Битрикса, если не получится разобраться буду костылями допиливать. Спасибо.

[Ипатьев]

текст с капслоком и без усваивается одинаково

людьми, которые умеют читать - да. Но для вас приходится выделять ключевые слова, чтобы вам было легче воспринимать текст. Вот теперь, после капслока, у вас получилось минимально связное описание проблемы

[Ilia Malashko]

Ипатьев, не написал что в download_page() выполняется curl.
Ситуация сейчас выглядет по другому.

$value = curl_exec($ch); // тут пусто
$data= curl_getinfo ($ch); // ответ ниже в споллере

curl_getinfo

[url] => https://wisell.ru/bitrix/catalog_export/megamarket.php
[content_type] =>
[http_code] => 0
[header_size] => 0
[request_size] => 0
[filetime] => -1
[ssl_verify_result] => 0
[redirect_count] => 0
[total_time] => 31.385636
[namelookup_time] => 0.000159
[connect_time] => 0
[pretransfer_time] => 0
[size_upload] => 0
[size_download] => 0
[speed_download] => 0
[speed_upload] => 0
[download_content_length] => -1
[upload_content_length] => -1
[starttransfer_time] => 0
[redirect_time] => 0
[redirect_url] =>
[primary_ip] =>
[certinfo] => Array
(
)

[primary_port] => 0
[local_ip] =>
[local_port] => 0
[http_version] => 0
[protocol] => 0
[ssl_verifyresult] => 0
[scheme] =>
[appconnect_time_us] => 0
[connect_time_us] => 0
[namelookup_time_us] => 159
[pretransfer_time_us] => 0
[redirect_time_us] => 0
[starttransfer_time_us] => 0
[total_time_us] => 31385636
[effective_method] => GET

Но если загрузить файл через wget то весть xml на месте

[Ипатьев]

Ну вот. Получается, что xml вообще не при чём.
Смотреть curl_error(), но скорее всего там просто таймаут, файл не успевает сгенерироваться.

Answer 1

[Ипатьев]

1. Гуглим simplexml_load_string ошибки
2. Открываем страницу https://www.php.net/manual/ru/simplexml.examples-e...
3. Берём с неё код и немного модицифируем

libxml_use_internal_errors(true);

$sXML = download_page($url);
$xml = simplexml_load_string($sXML);

if ($xml === false) {
    throw new RuntimeException(
        "XML parsing errors: ". implode("", array_column(libxml_get_errors(), 'message'))
    );
}

Смотрим ошибки, исправляем.