diff options
| author | srdusr <[email protected]> | 2025-11-16 22:17:00 +0200 |
|---|---|---|
| committer | srdusr <[email protected]> | 2025-11-16 22:17:00 +0200 |
| commit | 3af3e356d6fdf43e6772dc2e91b322f8e8148f62 (patch) | |
| tree | 3bc5697412a50818ef633caee151a01927b96053 /tests/test_filter.cpp | |
| parent | 407249eb5d654b5a951c43bc1722fd397d5d922c (diff) | |
| download | packeteer-3af3e356d6fdf43e6772dc2e91b322f8e8148f62.tar.gz packeteer-3af3e356d6fdf43e6772dc2e91b322f8e8148f62.zip | |
Add a cleartext-only QUIC dissector; fix a port-collision bug in L7Registry
QUIC decodes only what RFC 9000 sends in cleartext at the framing
level: long/short header form, version, long-packet type, and both
connection IDs. Everything past that is encrypted from the first
protected byte onward, even for Initial packets - decrypting that is
real crypto machinery this project deliberately doesn't take on, the
same call already made for TLS's SNI-only extraction.
Caught a real, previously-latent bug while wiring this in, by design
review rather than live-traffic debugging: L7Registry::dissect()
returned on the first dissector whose port() matched, even if that
dissector's summarize() then failed. Harmless while every registered
port was unique, but QUIC is the first protocol here to genuinely
share a well-known port with something already registered (443: TLS
over TCP, QUIC over UDP - the registry has no transport dimension).
Without the fix, tls_dissector would silently claim every port-443
lookup and QUIC would never be reachable. Fixed to try each same-port
dissector until one actually succeeds, locked in with stub-dissector
tests independent of any real protocol's parsing.
Live-verified thoroughly: real HTTP/3 traffic to google.com via
`curl --http3-only`, captured on wlp1s0, correctly decoding the full
connection lifecycle against Google's actual production QUIC
implementation - Initial packets (including a genuine connection ID
migration mid-handshake), Handshake packets, and 1-RTT short-header
packets. This also confirms the L7Registry fix live: without it none
of this would have decoded at all.
Diffstat (limited to 'tests/test_filter.cpp')
0 files changed, 0 insertions, 0 deletions