Сломались платежные системы после обновления. Почему не формируется PS_INVOICE_ID?

Question

[Анна]

Добрый день, коллеги! Снова обращаюсь к вам за помощью и очень надеюсь на дельный совет.
После обновления Битрикс от 03.02 на тестовом сайте сломались платежные системы, а точнее виджет ЮКассы с ответом от API - Неправильный запрос. Некорректное взаимодействие с API ЮKassa.

После звонка в тех поддержку и раскопок в коде я обнаружила, что в объекте оплаты параметр PS_INVOICE_ID остается пустым, в то время как на другом тестовом сайте (где все работает) этот параметр присутствует на том же этапе. Дальнейшее формирование запроса происходит уже некорректно.

Подскажите, пожалуйста, где как и где должен формироваться этот параметр? Буду очень благодарна за помощь!

Comments

[PetrPo]

Посмотри что в табличке b_sale_pay_system_err_log

[Анна]

PetrPo, Я указала в сообщении -Неправильный запрос. Некорректное взаимодействие с API ЮKassa.
Это я именно из таблицы взяла.

По словам тех поддержки приходит какой-то обрывочный запроc. Как будто где-то ошибка глупая в виде забытой скобки или чего-то похожего

[PetrPo]

Анна, напиши в консоли

\Bitrix\Main\Config\Option::set('sale', 'pay_system_log_level', 0);

Попробуй оплатить, потом посмотри в той же табличке будет что-то типа
Sale\Handlers\PaySystem\YandexCheckoutHandler: request data: какой_то_json

[Анна]

PetrPo,

Сделала, получила response-request-response в следующем виде

1 - Sale\Handlers\PaySystem\YandexCheckoutHandler: response data: { "account_id" : "739096", "test" : true, "fiscalization_enabled" : false, "payment_methods" : [ "yoo_money", "bank_card" ], "status" : "enabled" }

2 - Sale\Handlers\PaySystem\YandexCheckoutHandler: request data: {"description":"Оплата №97942\/1 заказа №97942 для [e-mail]","amount":{"value":2270,"currency":"RUB"},"capture":true,"metadata":{"BX_PAYMENT_NUMBER":23145,"BX_PAYSYSTEM_CODE":"4","BX_HANDLER":"YANDEX_CHECKOUT","cms_name":"api_1c-bitrix"},"confirmation":{"type":"embedded"}}

3 -Sale\Handlers\PaySystem\YandexCheckoutHandler: response data: { "type" : "error", "id" : "9ca7824a-75c8-4c8e-a078-e67c872bc1ee", "code" : "invalid_request", "description" : "Invalid request parameter", "parameter" : "confirmation" }

Не нравится параметр confirmation. Вроде в документации ЮКассы параметр для виджета так и выглядит, но не уверена. Ему чего-то не хватает?

[PetrPo]

Анна, мне к сожалению не на чем тестить, сам json корректный, по документации тоже посмотрел, вроде все верно.
Единственное мне не нравится, что в 1 - "payment_methods" : [ "yoo_money", "bank_card" ], а у тебя embedded, но тут ничего точно сказать не могу, может это норм, поэтому могу предложить:
1. Попробуй в битриксе пересохрани платежную систему
2. Отправить эти данные в юкассу и уточнить что им не нравится в этом запросе

[Анна]

PetrPo, пересохранила, даже другую создала с новыми настройками и та же беда. В тех поддержке сказали, что запрос обрывается по пути, и {"type":"embedded"} они уже не получают.

Отправила запрос техническим специалистам ЮКассы вместе с логами, может они что подскажут.

[Анна]

PetrPo,

Вот ответ от ЮКассы:

"Мне кажется в данном случае ваша система режет запрос на 259 символе и не отправляет нам запрос полностью.
Потому что теперь запрос обрывается на:
[2021-02-07T15:21:24.932000000+03:00] INFO body={"description":"Оплата №97944\/1 заказа №97944 для [e-mail]","confirmation":{"type":"embedded"},"amount":{"value":2190,"currency":"RUB"},"capture":true,"metadata":{"BX_PAYMENT_NUMBER":23147,"BX_PAYSYSTEM_CODE":"14","BX_HANDLER":"YANDEX_CHECKOUT","cms_name

Уточните, пожалуйста, данный вопрос у разработчика модуля или проверьте валидатор входящих параметров для отправки запроса в ЮKassa."

Не могли бы вы подсказать, пожалуйста, почему может резаться запрос на Битрикс и где это можно исправить? Я как-то раньше не сталкивалась с таким.

[PetrPo]

Анна, врядли конечно в этом дело, но в php max_input_vars сколько выставлен?

[Анна]

PetrPo,
max_input_vars в Local value установлен 10000, master -1000
Из ошибок системы у нас только - Лишний вывод в файлах конфигурации.
Может ли что-то из этого повлиять на обрезание запроса?

[PetrPo]

Анна, теоретически может, но если сломалось после обновления то мало вероятно. А в init.php есть что-нибудь?

[Анна]

PetrPo, вроде нет. Вы имеете в виду лишние пробелы?

[Анна]

PetrPo, я проверяла dbconn и init.php (в нем нет закрывающего тега <?>). Вроде никаких проблем нет

[PetrPo]

Анна, не знаю даже, странно как-то. Можно попробовать подебажить отправку запроса. В файле /bitrix/modules/main/lib/web/httpclient.php есть метод

protected function sendRequest($method, Uri $url, $entityBody = null)
{
//..................

	$request .= $this->requestHeaders->toString();
	$request .= "\r\n";

	if(is_resource($entityBody))
	{
		//PUT data can be a file resource
		while(!feof($entityBody))
		{
			$this->send(fread($entityBody, self::BUF_POST_LEN));
		}
	}
	elseif($entityBody <> '')
	{
		$this->send($entityBody);
	}
}

Вот после $request .= "\r\n"; посмотреть что в заголовках - принтануть $request, там должен быть Content-Length и сравнить совпадает ли число с длиной json строки. И посмотреть саму строку - переменная $entityBody, не обрезалась ли она.

[Анна]

PetrPo,

request : POST /v3/payments HTTP/1.0
Content-Type: application/json
Idempotence-Key: ****(не привожу оригинал)
Authorization: Basic ***(не привожу оригинал)
Host: api.yookassa.ru
Connection: close
Accept: */*
Accept-Language: en
Content-Length: 277

$entityBody : {"description":"Оплата №97946\/1 заказа №97946 для [e-mail]","amount":{"value":2270,"currency":"RUB"},"capture":true,"metadata":{"BX_PAYMENT_NUMBER":23149,"BX_PAYSYSTEM_CODE":"4","BX_HANDLER":"YANDEX_CHECKOUT","cms_name":"api_1c-bitrix"},"confirmation":{"type":"embedded"}}

Длина совпадает. Я еще вычистила все ошибки системы, на всякий пожарный и подняла показатели на сервере для post. Лучше не стало.

[PetrPo]

Анна, ну дальше уже некуда, там вызываются php функции fwrite, затем fgets, т.е. это не битрикс режет запрос. Проверь еще в настройках платежной системы кодировка правильная указана? и сервер случайно не локальный?

[Анна]

PetrPo, нет, это тестовый сайт, но не локалка. На локальной копии проблем вообще нет.
Кодировки все попробовала, ноль эмоций.
Жду ответа от тех поддержки Битрикса, посмотрим, что они скажут. Пока ЮКасса сообщает, что проблемы на нашей стороне.

[PetrPo]

Анна, напиши, если что-нибудь выяснится )

[Анна]

PetrPo, Обязательно отчитаюсь)

[PetrPo]

Анна, почитал про функцию fwrite, вроде как она не гарантирует передачу всех данных и сетевое соединение может закрыться раньше времени. Попробуй все в той же функции

//.............
elseif($entityBody <> '')
  {
    $length = $this->send($entityBody);
  }

в $length должна быть цифра равная Content-Length

[Анна]

PetrPo, Странно, но у меня эта цифра больше.
$request : POST /v3/payments HTTP/1.0
Content-Type: application/json
Idempotence-Key: ***
Authorization: Basic ***
Host: api.yookassa.ru
Connection: close
Accept: */*
Accept-Language: en
Content-Length: 280

$length : 299

[PetrPo]

Анна, php какая версия?

[Анна]

PetrPo, 7.3

[PetrPo]

Анна, параметр php mbstring.func_overload сколько выставлен? должен быть 0

UPD
Собственно, причину почему юкасса получает обрезанный запрос, мы нашли) Сообщение длинее чем указано в заголовке.
Судя по коду причина одна, несовпадение при подсчете длины строки функцией strlen($entityBody) и записанных байтов функцией fwrite, вероятнее всего из-за func_overload, т.к. с версии php 7.2 это дело deprecated

[Анна]

PetrPo,
Но у нас это значение установлено 0. Битрикс выдавал ошибки при проверке системы и я это первым делом пофиксила и ошибка ушла. Есть только момент, что я установила это значение только для указанного сайта. Мастер значение осталось 2. Может ли быть такое что мастер значение перекрывает локальное значение и от этого режется запрос?

[PetrPo]

Анна,

Может ли быть такое что мастер значение перекрывает локальное значение

На счет этого не знаю, в серверной части не силен. Если есть возможность мастер поменять, проверь)

UPD
ну да, все сходится
Оплата №97946\/1 заказа №97946 для [e-mail]
15 символов на кириллице по 1байту вместо 2байтов, 2раза знак № по 1байту вместо 3байтов
Итого: 15*(2-1) + 2*(3-1) = 19байт

М.б. еще какой-то параметр php за это отвечает, но func_overload точно должен быть 0 и возможно php его из мастера берет, как сказано в документации:

Только INI-настройки с режимами PHP_INI_PERDIR и PHP_INI_USER будут распознаны в INI-файлах в стиле .user.ini.

но вот какой режим у func_overload фиг знает)

[PetrPo]

Анна, нашел все-таки
mbstring.func_overload режим PHP_INI_SYSTEM - из документации значение может быть установлено в php.ini или httpd.conf, т.е. локально установить нельзя, надо чтобы на мастере было 0

[Анна]

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

[Анна]

PetrPo, Вы абсолютно правы!!! Не смотря на то, что локальные параметры вроде имеют приоритет, но нам реально все ломало это значение в мастере. Теперь все работает, спасибо вам огромное!
Можете вынести отдельно ответ, я его отмечу как верное решение?

[PetrPo]

Анна, пожалуйста, не забудь удалить из b_option запись про pay_system_log_level

[Анна]

PetrPo, Да, конечно! Еще раз большое спасибо. Решением отметила

Answer 1

[PetrPo]

Проблема была в том, что мастер значение mbstring.func_overload = 2, из-за этого неправильно указывался заголовок Content-Length и запрос в юкассу обрезался. Подробнее в комментариях.

Логи по платежным системам в битриксе можно смотреть в таблице b_sale_pay_system_err_log. Если установить опцию через консоль будут записываться логи уровня debug

\Bitrix\Main\Config\Option::set('sale', 'pay_system_log_level', 0);