Skip to content

bug: PHP Fatal error: Uncaught Exception: Error occurred during parsing: Error parsing JSON @1:9: No such field: @type in RequestTrait.php #149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
vladimirfokingithub opened this issue Jan 30, 2025 · 5 comments · Fixed by #150
Labels
bug Something isn't working

Comments

@vladimirfokingithub
Copy link

Bug Report

YDB PHP SDK version: 1.15

bug: PHP Fatal error: Uncaught Exception: Error occurred during parsing: Error parsing JSON @1:9: No such field: @type in RequestTrait.php

Решается изменением
$result->mergeFromJsonString($jsonResult); -> $result->mergeFromJsonString($jsonResult, true);

@vladimirfokingithub vladimirfokingithub added the bug Something isn't working label Jan 30, 2025
@rekby
Copy link
Member

rekby commented Feb 3, 2025

@vladimirfokingithub, здравствуйте. Спасибо за сообщение, можно вас попросить пример кода на котором воспроизводится проблема?

И можно ли вас попросить оформить pr с исправлением?

@a-brazhnikov
Copy link
Contributor

a-brazhnikov commented Feb 13, 2025

Здравствуйте. У меня такая же проблема воспроизводится если для php установлен ext-protobuf v3.25.6 (пришлось установить, чтобы поднять производительность).

Пример кода, при котором возникает данная ошибка:

$result = $ydb->table()->retryTransaction(function(Session $session) use($query, $params) {
       return $session->prepare($query)->execute($params);
}, true);

Действительно решилось этой правкой vendor/ydb-platform/ydb-php-sdk/src/Traits/RequestTrait.php:
$result->mergeFromJsonString($jsonResult); -> $result->mergeFromJsonString($jsonResult, true);

@rekby
Copy link
Member

rekby commented Feb 14, 2025

@a-brazhnikov спасибо за подтверждение.

Я правильно понимаю, что на ext-protobuf v3.25.6 эта проблема наблюдается просто при выполнении любого запроса?

И на всякий случай - подскажите версию php.

И вы можете сделать PR или подождёте, пока я доберусь всё проверить и поправить?

@vladimirfokingithub
Copy link
Author

php8.4, protobuf последний, но насколько я понял он уже давно не обновляется.
pr постараюсь сделать сегодня-завтра.

@rekby rekby linked a pull request Mar 17, 2025 that will close this issue
7 tasks
@rekby rekby closed this as completed Mar 17, 2025
@rekby
Copy link
Member

rekby commented Mar 17, 2025

PR влил, спасибо за фикс и прости за долгий процесс

Версия с исправлением опубликована - 1.15.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants