-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
server: doesn't send DL_VALUES if there are a lot of settings #1344
Comments
@gautierhattenberger @alex31 any idea why there is this limitation? |
Actually, Ivy should support long messages since 2008, but Pascal added this check in 2009. We should try to remove it and see because their is no reasons to keep it from what I can see. |
So what is the maximum length now? |
there is an initial malloc and a realloc if needed, so I guess the limit is the available memory... but yes, limiting stupidly long message is probably wise |
So we limit it to 4000 or 10000 or so? |
tested and working |
When you add more settings, at some point server doesn't send DL_VALUES anymore with
Discarding long ivy message DL_VALUES (1240 bytes)
What is this Ivy bug the limit of 1000 chars in
message_send
in pprz.ml is supposed to prevent?The text was updated successfully, but these errors were encountered: