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.

Sign up for GitHub

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

Jump to bottom

Handle more TCP edge cases #61

Merged
merged 13 commits into from
Dec 16, 2024
Merged

Handle more TCP edge cases #61

merged 13 commits into from
Dec 16, 2024

Conversation

hamishcoleman
Copy link
Contributor

A number of scenarios in the TCP handling were not fully handled. Find those and add appropriate handling for them.

Also:

  • Fix CI pipeline tests in the face of incompatibilities caused by github runner upgrades
  • Add an example podman containerfile to demonstrate generating docker images

@hamishcoleman
Allow tracking how many bytes are read into a conn struct
6834db5
@hamishcoleman
Just because a read would block, doesnt make the conn empty
8815dc5
@hamishcoleman
Deprecate the use of the CONN_SENDING state to allow easier bi-direct…
6648dcb 
…ional traffic
@hamishcoleman
Just because we have finished writing doesnt make the read side empty
5f63b46
@hamishcoleman
Ensure that the non blocking states are handled in the write path (an…
b274360 
…d dont decrement the sendpos)
@hamishcoleman
Increase readability by clearly showing the ITER basic block as indented
b5ef528
@hamishcoleman
Update user visible message to not claim UDP - both TCP and UDP are p…
9810e9e 
…rocessed here
@hamishcoleman
Avoid trying to process the release of an error fd
d0b35e5
@hamishcoleman
Clearly log errors with some setsockopts
71d4e5c
@hamishcoleman
Ensure we do not ignore the return value for a write
99ffa5e
@hamishcoleman
Use a specified OS version for all lint tests
0d4d19b 
The switch to a newer ubuntu-24.04 also imported a set of lint tests
that do not match anything the dev systems are using.
@hamishcoleman
Add an example container build
021d831
@hamishcoleman
Finish removal of CONN_SENDING state
73d198a
@hamishcoleman hamishcoleman self-assigned this Dec 16, 2024
@hamishcoleman hamishcoleman merged commit 91c05f0 into n42n:main Dec 16, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hamishcoleman hamishcoleman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hamishcoleman