grammyjs / menu

Interactive menus for grammY.
https://grammy.dev/plugins/menu
MIT License
31 stars 11 forks source link

feat: check payload data length and throw exception when it exceeds limit #46

Open ubertao opened 10 months ago

ubertao commented 10 months ago

I got a 400 Error BUTTON_DATA_INVALID when using menu plugin. The error message is very confusing and it took me a while to debug.

Turns out it's because I used long menu id and and long payload data so the callback query data exceeded Telegram's limit. (I didn't know menu id was also passed to api as part of callback query data. And even if I knew, it would be tricky for developer to keep track of bytes used for callback query data because menu plugin adds its own stuff to it).

Would be helpful if menu plugin could check payload length before making api request, and output some meaningful error message, e.g. telling me how many bytes are available for payload data.

KnorpelSenf commented 5 months ago

This should ideally just be a warning. I do not want to perform client-side validation because the limits may change and then the plugin would be more restrictive than necessary.

DonVietnam commented 2 months ago

This should ideally just be a warning. I do not want to perform client-side validation because the limits may change and then the plugin would be more restrictive than necessary.

Just yesterday, I spent more than 5 hours searching for the reason why I was getting this error, and I got closer to the answer only after reading this page, so it seems that this is a really significant problem, the error should be more informative, and at the moment the message leads away from the real problem.

KnorpelSenf commented 2 months ago

That is because the error is created by Telegram not grammY. We also should not validate/error out in grammY because that might break forward compatibility. Hence, adding such a warning is what needs to be implemented. @DonVietnam do you feel like contributing this to the plugin?