userver icon indicating copy to clipboard operation
userver copied to clipboard

Published 20 hours ago • verified publisher icon userver-framework

limit json depth to 128

Open moki opened this issue 1 year ago • 4 comments

Closes: #450

moki avatar Dec 09 '23 15:12 moki

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

github-actions[bot] avatar Dec 09 '23 15:12 github-actions[bot]

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/

moki avatar Dec 09 '23 19:12 moki

Tests are failing!

Please look at userver/universal/src/formats/json/impl/types_impl.cpp (yes, a tricky one) how rapidjson's Value must be destroyed to avoid stack overflow.

segoon avatar Dec 11 '23 06:12 segoon

Tests are failing!

Please look at userver/universal/src/formats/json/impl/types_impl.cpp (yes, a tricky one) how rapidjson's Value must be destroyed to avoid stack overflow.

Will do, thanks for the tip!

moki avatar Dec 11 '23 17:12 moki