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

Implement PayloadLoad operation (parsing, interpretation, evaluation, tests). #10782

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Aug 16, 2024

Implement PayloadLoad operation (parsing, interpretation, evaluation, tests).

Tests payload load evaluation for all basic fields of IP, IPv6, & TCP headers.
These are the main headers that are encountered for gVisor, so their
implementation was prioritized. Loading for other packet headers should work
as expected since raw/general payload loading works, but it has not been
explicitly tested. These tests should be added later and a TODO has been left
noting this.

@copybara-service copybara-service bot added the exported Issue was exported automatically label Aug 16, 2024
@copybara-service copybara-service bot force-pushed the test/cl662226011 branch 2 times, most recently from 879c9e4 to c049504 Compare August 27, 2024 23:27
… tests).

Tests payload load evaluation for all basic fields of IP, IPv6, & TCP headers.
These are the main headers that are encountered for gVisor, so their
implementation was prioritized. Loading for other packet headers should work
as expected since raw/general payload loading works, but it has not been
explicitly tested. These tests should be added later and a TODO has been left
noting this.

PiperOrigin-RevId: 668202252
@copybara-service copybara-service bot merged commit 33dc938 into master Aug 28, 2024
@copybara-service copybara-service bot deleted the test/cl662226011 branch August 28, 2024 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exported Issue was exported automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants