-
-
Notifications
You must be signed in to change notification settings - Fork 356
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
max_datagram_size() seems to be off by 2 #1412
Comments
Thanks for the report! We should really have a unit test for this. |
I'm not able to reproduce this on 0.8 or main. Can you provide a repro? |
I'll try, but my first attempt at a repro for the assertion bug didn't yield the same problem the original code producing this issue is, haven't found what the difference between the 2 is so far. |
I had a similar issue where when i sent a packet with a size of max_datagram_size it wouldn't reach its destination, but when subtracting 1 or more from it, it worked |
Closing as not reproducible. |
Not sure if this should just be documented or if there should be a fix to the calculation.
If I try to send a packet via
send_datagram
of data at the size returned bymax_datagram_size()
I'll get aToolarge
error. I've had to reduce it by 2 to get it to pass the check & send the datagram.I'm
calling max_datagram_size()
at time of data generation & immediatesend_datagram()
call so it should be current.The text was updated successfully, but these errors were encountered: