Skip to content
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

The reader's MaxDepth of 64 has been exceeded #1618

Open
ivanmem opened this issue Jul 19, 2024 · 3 comments
Open

The reader's MaxDepth of 64 has been exceeded #1618

ivanmem opened this issue Jul 19, 2024 · 3 comments
Labels
Задача Задача требующая реализации нового или доработке существующего функционала
Milestone

Comments

@ivanmem
Copy link
Contributor

ivanmem commented Jul 19, 2024

Когда на сообщение отвечено более 64х раз при вызове метода Messages.GetByConversationMessageId вылетает ошибка:

"Newtonsoft.Json.JsonReaderException", "Message": "The reader's MaxDepth of 64 has been exceeded.

Есть минимум три варианта решения:

  1. реализовать настройку сериализатора, наподобие такого:

image
2) поменяйте конфигурацию в вкнет до MaxDepth =128 или сколько нужно (не знаю точно)
3) смените MaxDepth конкретно для метода GetByConversationMessageId

@inyutin-maxim inyutin-maxim added the Задача Задача требующая реализации нового или доработке существующего функционала label Aug 28, 2024
@inyutin-maxim inyutin-maxim added this to the v1.79.0 milestone Aug 28, 2024
@CaCTuCaTu4ECKuu
Copy link
Member

А это вообще как бы, ну, реалистичный use-case??
Сколько не поднимай всегда можно сделать больше.

Т.к. у нас все типизировано и ВК тоже отдает адекватные данные то в принципе можно выставить довольно большое значение глубины по дефолту, посмотрю потом, добавлю чтобы можно было у VkApi менять стерилизатор

@ivanmem
Copy link
Contributor Author

ivanmem commented Oct 4, 2024

ну, реалистичный use-case??

Редкий случай, но да, реалистичный. Я хотел сохранять весь контекст в чат боте с помощью ответа на предыдущее сообщение. Вот в таких случаях бывает и больше 100 сообщений. Но 200 и тд - вряд ли будет.

@CaCTuCaTu4ECKuu
Copy link
Member

Где есть 100 там и 200 будет, я понял
Ну тогда в конфиг какой-то добавлю чтоб можно было переопределить, а кто сломает сам дурак

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Задача Задача требующая реализации нового или доработке существующего функционала
Projects
None yet
Development

No branches or pull requests

3 participants