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(ebpf): udp problem caused by #221 by accident #225

Merged
merged 1 commit into from
Jul 16, 2023

Conversation

mzz2017
Copy link
Contributor

@mzz2017 mzz2017 commented Jul 16, 2023

Background

In #221, I incorrectly parse UDP header as TCP header, which causes a failure in parsing five tuples and results in bad connectivity of UDP.

Checklist

Full changelogs

  • Correct UDP parsing.

Issue reference

Closes #224

@mzz2017
Copy link
Contributor Author

mzz2017 commented Jul 16, 2023

Test result:
图片

Copy link
Contributor

@dae-prow dae-prow bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧪 Since the PR has been fully tested, please consider merging it.

@mzz2017 mzz2017 merged commit 936b2a0 into main Jul 16, 2023
22 checks passed
@mzz2017 mzz2017 deleted the fix_ebpf_udp_mapping branch July 16, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken UDP caused by #221
1 participant