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

Fix response packet size #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix response packet size #1

wants to merge 3 commits into from

Conversation

orn1983
Copy link

@orn1983 orn1983 commented Nov 8, 2022

The size of the response packet was calculated by checking the length of the receive buffer. The buffer was created with a fixed size of 10240, so that was always the size of the response payload.

Now we store the value of bytes read from ReadFromUDP and pass it on to createTestResponse so that we can create a response packet of the same size as the request packet.

The size of the response packet was calculated by checking the length of
the receive buffer. The buffer was created with a fixed size of 10240,
so that was always the size of the response payload.

Now we store the value of bytes read from ReadFromUDP and pass it on to
createTestResponse so that we can create a response packet of the same
size as the request packet.
UDP timeouts with trigger a blocking channel read to check if the test
was done, whereas the intention was to check if the test was done, and
if not, move on and start waiting again. That has now been implemented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant