Skip to content

Implementing QUIC v2 support and added Connection Close frame#4939

Open
gilkzxc wants to merge 1 commit intosecdev:masterfrom
gilkzxc:patch-1
Open

Implementing QUIC v2 support and added Connection Close frame#4939
gilkzxc wants to merge 1 commit intosecdev:masterfrom
gilkzxc:patch-1

Conversation

@gilkzxc
Copy link

@gilkzxc gilkzxc commented Feb 28, 2026

  • Fix support for parsing QUIC Packets in version 2, and not just version 1. By implementing correct hash-table of packet types to fit into the correct types by RFC9639 sect 3.2. As well the correct matching type classes, by modifying QUIC class.

  • Added Connection Close frame implementation. As stated RFC9000 sect 19.19

    • This code has been modified as part of a different project mine in QUIC, I have tested it to some degree.

Checklist:

  • If you are new to Scapy: I have checked CONTRIBUTING.md (esp. section submitting-pull-requests)
  • I squashed commits belonging together
  • I added unit tests or explained why they are not relevant
  • I executed the regression tests (using tox)
  • If the PR is still not finished, please create a Draft Pull Request

fixes #xxx

- Fix support for parsing QUIC Packets in version 2, and not just version 1. By implementing correct hash-table of packet types to fit into the correct types by RFC9639 sect 3.2. As well the correct matching type classes, by modifying QUIC class.

- Added Connection Close frame implementation. As stated RFC9000 sect 19.19
@codecov
Copy link

codecov bot commented Feb 28, 2026

Codecov Report

❌ Patch coverage is 43.90244% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.84%. Comparing base (bba4ec0) to head (047c84d).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
scapy/layers/quic.py 43.90% 23 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (bba4ec0) and HEAD (047c84d). Click for more details.

HEAD has 10 uploads less than BASE
Flag BASE (bba4ec0) HEAD (047c84d)
11 1
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4939       +/-   ##
===========================================
- Coverage   80.11%   44.84%   -35.27%     
===========================================
  Files         370      352       -18     
  Lines       91917    91621      -296     
===========================================
- Hits        73640    41091    -32549     
- Misses      18277    50530    +32253     
Files with missing lines Coverage Δ
scapy/layers/quic.py 50.73% <43.90%> (-40.53%) ⬇️

... and 328 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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