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

[Agent] Optimizes HTTP2 parsing headers #5035

Merged
merged 2 commits into from Dec 28, 2023
Merged

[Agent] Optimizes HTTP2 parsing headers #5035

merged 2 commits into from Dec 28, 2023

Conversation

TomatoMr
Copy link
Contributor

This PR is for:

  • Agent

Optimizes HTTP2 parsing headers

Steps to reproduce the bug

Changes to fix the bug

  • Fixed panic when parsing unexpected http2 payload
  • Reduce http2 dynamic table size by add a header filter

Affected branches

  • main
  • v6.3

Checklist

  • Added unit test to verify the fix.
  • Verified eBPF program runs successfully on linux 4.14.x.
  • Verified eBPF program runs successfully on linux 4.19.x.
  • Verified eBPF program runs successfully on linux 5.2.x.

@TomatoMr TomatoMr force-pushed the fix_http2_header_main branch 7 times, most recently from 2b4bace to a002d7a Compare December 28, 2023 04:38
@TomatoMr TomatoMr marked this pull request as ready for review December 28, 2023 04:38
@rvql rvql enabled auto-merge (rebase) December 28, 2023 06:19
@rvql rvql merged commit 6a3c0a7 into main Dec 28, 2023
7 checks passed
@rvql rvql deleted the fix_http2_header_main branch December 28, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rvql rvql rvql approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@TomatoMr @rvql