http3: add basic server-side qlog support (#5367)

* add Conn.QlogTrace

* http3: add basic qlog support on the server side

ci: include OS and Go version in Codecov test report upload (#5370)

fix qlogging of the short header payload length (#5365)

qlog: privatize Encode methods of non-Event structs (#5364)

ackhandler: fix qlogging of alarm timer expiration time (#5363)

qlogwriter: pass the event time to Event.Encode (#5362)

This is needed for events such as recovery:loss_timer_updated, which
contain the timer expiration timestamp encoded as a difference from the
event time.

qlogwriter: add support for event_schemas in the trace header (#5361)

qlogwriter: implement the draft-12 trace header (#5360)

qlog: split serializiation and event definitions, remove logging abstraction (#5356)

* qlog: implement a Trace and a Writer struct

* qlog: rename Trace to FileSeq

* split qlog trace writer and QUIC qlog events into separate packages

* use the new qlog.Recorder instead of the logging.ConnectionTracer

use synctest in more connection tests (#5357)

No functional change expected.

ci: remove 386 (32 bit x86) (#5352)

The last 386 cpu was from 20 years ago (Pentium 4)

qlog: implement a minimal jsontext-like JSON encoder (#5353)

* qlog: use fork of encoding/json/jsontext instead of unmaintained gojay

* implement a minimal jsontext-compatible encoder

* qlogtext: improve fuzz test

* qlog: simplify JSON encoding error handling

* qlog: make use of jsontext.Bool

ackhandler: track lost packets and detect spurious losses (#5355)

* ackhandler: implement a lost packet tracker

* logging: add ConnectionTracer.DetectedSpuriousLoss event

* ackhandler: detect spurious losses

* qlog: add support for logging spurious packet loss

* ackhandler: delete lost packets after detecting spurios loss

* guard against slice index underflow

fix missing tracing of restored transport parameters (#5349)

protocol: optimize ConnectionID.String (#5351)

This function is used by qlog, so it should be fast.

drop initial keys when the handshake is confirmed (#5354)

use synctest to make the connection tests fully deterministic (#5317)

improve connection timer logic (#5339)

* simplify timer resets

As of Go 1.23, timer resets work as expected, and there’s no need to
read a (potentially) stale value from the timer channel.

* don't save the last time

* track if blocked

* remove unblock after sendingScheduled

* fix unblock logic when sending

* don't use fallthrough

* track the blocking mode

* remove stale comment

add a benchmark test for data transfers (#5335)

implement a memory-optimized time.Time replacement (#5334)

* implement a memory-optimized time.Time replacement

* monotime: properly handle systems with bad timer resolution (Windows)

* monotime: simplify Since

ackhandler: add a benchmark for sending and acknowledging packets (#5333)

* ackhandler: add a benchmark for sending and acknowledging packets

* report allocs, improve benchmark names

return stream frames to pool on stream cancellations (#5327)

* fix: return stream frames to pool on error paths

fixes potential memory leak where StreamFrames were not returned to
sync.Pool in error/cancellation paths.

root cause: frames are llocated via GetStreamFrame() but never returned
via PutBack() when streams enter terminal states without going through
normal OnAcked() cleanup.

fixed paths:
- CancelWrite(): return frames when reliableOffset == 0
- handleStopSendingFrame(): return frames on STOP_SENDING
- closeForShutdown(): return frames on connection shutdown
- OnLost(): return frame when stream reset with no data sent

* fix: clear slice before nil assignment, remove excessive tests

- add clear() call to properly release GC references per https://github.com/quic-go/quic-go/pull/5327#discussion_r2328451862
- remove excessive pool return tests per https://github.com/quic-go/quic-go/pull/5327#discussion_r2328453227

ci: bump actions/setup-go from 5 to 6 (#5330)

Bumps [actions/setup-go](https://github.com/actions/setup-go) from 5 to 6.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](https://github.com/actions/setup-go/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

http3: don't allow usage of closed Transport (#5324)

* do not allow use a closed transport

* update tests

* add test

* Update http3/transport.go

Co-authored-by: Marten Seemann <martenseemann@gmail.com>
* unit tests

* update test

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

qlog: merge event category and name (#5329)

This follows a (not so recent) change to the qlog specification.

qlog: add a benchmark for the ConnectionTracer (#5328)

This test aims to benchmark a somewhat realistic scenario of a QUIC
connection that sends packets and receives acknowledgments.

http3: fix race in concurrent Transport.RoundTrip calls (#5323)

* http3: fix data race in Transport

* reset init Once

ackhandler: store the last four skipped packets (#5322)

This allows for more accurate packet number difference calculations,
which is especially important once we attempt to detect spurious loss
detection events.

ackhandler: account for skipped packets in packet threshold calculation (#5316)

ackhandler: store skipped packet numbers separately (#5314)

ackhandler: avoid storing packet number in packet struct (#5312)

* ackhandler: optimize memory layout of packet struct

The packet number can be derived from the position that this packet is
stored at in the packets slice in the sent packet history. There is no
need to store the packet number, saving 8 bytes per packet.

* ackhandler: avoid copying the packet struct

ci: fix cache paths in cross compile workflow (#5318)

ci: improve cache key generation for the cross compilation job (#5315)

ackhandler: immediately clear ackedPacket slice after processing ACK (#5313)

* ackhandler: immediately clear ackedPacket slice after processing ACK

* ackhandler: add defensive check for empty ackedPackets slice

ackhandler: use a typed mock for the ECNHandler (#5311)

No functional change expected.

ackhandler: use an iterator to process received packet ranges (#5309)

* ackhandler: use an interator to track received packet ranges

* use slices.Collect

* naming convention

ackhandler: fix resetting of packet.isPathProbePacket (#5310)

utils: use synctest to make the timer tests fully deterministic (#5306)

* utils: use synctest to make the timer tests fully deterministic

* Go 1.24 workaround

use synctest to make the datagram queue tests fully deterministic (#5305)

use synctest to make the outgoing path manager tests fully deterministic (#5304)

ci: use go mod tidy -diff to check for tidied go.mod (#5303)

use synctest to make the send stream tests fully deterministic (#5298)

use synctest to make the send queue tests fully deterministic (#5302)

use synctest.Wait in receive stream tests (#5299)

restore previously deleted TestStreamsMapConcurrent test (#5301)

ci: fix cache save and restore logic for cross compile workflow (#5300)

lookup-only doesn't download the cache, even if it's there. This is not what we
intended: we still want to use the cache for PRs, we just want to upload a
new cache.

ci: cache the Go build cache for cross-compilation workflow (#5297)

* ci: cache the Go build cache for cross-compilation workflow

* only store cache when on master

* don't hash go.sum in

use synctest to make streams map tests fully deterministic (#5296)

* use synctest to make streams map tests fully deterministic

* ignore synctest helper package for code coverage

use synctest to make receive stream tests fully deterministc (#5291)

ci: bump actions/checkout from 4 to 5 (#5293)

Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

use testing.B.Loop in all benchmark tests (#5285)

go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -category=bloop -fix -test ./...
was used as a starting point.

replace interface{} with any (#5290)

No functional change expected.

update to Go 1.25, drop Go 1.23, use go tool for gomock (#5283)

* update to Go 1.25, drop support for Go 1.23

* ci: update golangci-lint to v2.4.0

* use new go tool command for gomock / mockgen

* also update Go version used for oss-fuzz

ackhandler: generalize check for missing packets below threshold (#5260)

* ackhandler: check for missing packets below reordering treshold

No functional change expected.

With the Acknowledgement Frequency extension, the reordering threshold
will become configurable. With this change, it will be easy to use the
peer-requested value instead of the predefined constant.

* improve documentation

* call HighestMissingUpTo in randomized test

expose basic connection stats via Conn.ConnectionStats (#5281)

* Add ConnectionStats

* remove for loop

* Add comments

* Update comments

---------

Co-authored-by: Marco Munizaga <git@marcopolo.io>

fix documentation for EarlyListener.Close (#5284)

* quic: update documentaion for EarlyListener.Close

* fix missing space

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

http3: fix documentation for Server.ServeListener (#5282)

fix repeated initialization of outgoingPathManager in Conn.AddPath (#5280)

In case of concurrent calls, this may still be called multiple times.

Co-authored-by: Marco Munizaga <git@marcopolo.io>

quicvarint: tolerate empty reads of the underlying io.Reader (#5275)

* Tolerate empty reads in `quicvarint.Read`

Currently, `quicvarint.Read` misinterprets an empty read (`n==0`)
as a single zero byte.  While empty reads are "discouraged" in
the `io.Reader` interface, they do sometimes happen, especially
when using `io.Pipe()`.

This change tolerates empty reads, adds a corresponding unit test,
and also includes some additional test coverage related to empty
capsules

* minor test refactor

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

quicvarint: extend benchmark to use quicvarint.Reader (#5278)

* quicvarint: extend benchmark to use quicvarint.Reader

* rename variable

ackhandler: remove unused time from receivedPacketHandler.ReceivedPacket (#5277)

* ackhandler: remove unused time from receivedPacketHandler.ReceivedPacket

No functional change expected.

* fix typo

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* also fix typo in tests

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

qlog: add support for ACK_FREQUENCY and IMMEDIATE_ACK frames (#5276)

fix missing log statement for STREAM, DATAGRAM and ACK (#5273)

wire: add support for the min_ack_delay transport parameter (#5266)

* wire: implement parsing and writing of the min_ack_delay transport parameter

* wire: validate the min_ack_delay transport parameter

fuzzing: fix timeout in frame parser (#5268)

wire: implement parsing and writing of the IMMEDIATE_ACK frame (#5265)

wire: implement parsing and writing of the ACK_FREQUENCY frame (#5264)

* implement parsing and writing of the ACK_FREQUENCY frame

* prevent overflows of Request Max Ack Delay

* fix fuzzer

wire: optimize parsing logic for STREAM, DATAGRAM and ACK frames (#5227)

ParseOtherFrames-16       148ns ± 4%     150ns ± 3%     ~     (p=0.223 n=8+8)
ParseAckFrame-16          302ns ± 2%     298ns ± 3%     ~     (p=0.246 n=8+8)
ParseStreamFrame-16       262ns ± 3%     213ns ± 2%  -18.61%  (p=0.000 n=8+8)
ParseDatagramFrame-16     561ns ± 5%     547ns ± 4%     ~     (p=0.105 n=8+8)

wire: improve frame parsing benchmarks (#5263)

* Add master-style frame handling benchmarks using type switches

* Fixing styling issue.

* put STREAM frame back

* remove BenchmarkParseStreamAndACK

* use random data for STREAM and DATAGRAM

* improve comment

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

close Transport when DialAddr fails (#5259)

Close the transport after dial fails to avoid memory leaks. Same logic as DialAddrEarly.

Signed-off-by: Coia Prant <coiaprant@gmail.com>

http3: remove dependency on quic internal packages (#5256)

* Remove http3 dependency on quic internal packages

Remove the dependency on internal/protocol from the http3 package. This
makes it possible for a forked http3 to use the mainline quic-go
package.

* Address review comments

* Fix syntax

* Use broader pattern for http3 directory

* Copy internal/testdata

* Replace perspective with bool

* clone the supported version slice

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

fix flaky TestPostQuantumClientHello (#5253)

add a Config and ConnectionState flag for RESET_STREAM_AT (#5243)

* add a Config and ConnectionState flag for RESET_STREAM_AT

* add RESET_STREAM_AT to README

http3: fix flaky TestConnGoAwayFailures (#5252)

fix flaky TestServerReceiveQueue (#5249)

fix flaky TestDrainServerAcceptQueue (#5247)

The connections are not necessarily accepted in the same order that
they are dialed.

fix flaky TestTransportReplaceWithClosed (#5245)

implement sender side behavior for RESET_STREAM_AT (#5242)

* improve existing send stream test

* implement sender side behavior for RESET_STREAM_AT

* refactor send stream cancelation and shutdown error handling

* correctly deal with 0-RTT corner case

implement receiver side behavior for RESET_STREAM_AT (#5235)

* implement receiver side behavior for RESET_STREAM_AT

* simplify reliable offset tracking

fix retransmission logic for path probing packets (#5241)

To achieve an exponential backoff, the timer should only be reset after
having fired.

quicvarint: speed up parsing of 1, 2 and 4 byte varints  (#5229)

drain server accept queue when the transport is closed (#5237)

* drain server queue on transport close

* add integration test for clearing conn queue

* improve documentation and test for Transport.Close

* move to handshake_test.go

---------

Co-authored-by: sukun <sukunrt@gmail.com>

fix deadlock when closing the Transport (#5220)

* transport: fix deadlock on close

The lock ordering is Transport.mutex before connMx. This deadlocks
some times in tests.
For the latest one:
https://github.com/libp2p/go-libp2p/actions/runs/15534001571/job/43728863965?pr=3305

* remove connMx

* release mutex for server close

* Update server.go

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>

handshake: store key update interval in an atomic (#5234)

* handshake: store key update interval in an atomic

We recently changed the way the key update interval is set in tests to
use an environment variable. This resolved a race condition that existed
in the earlier logic, however, parsing of the environment variable now
shows up in benchmark tests.

Using an atomic variable should have a negligible performance impact.

* use an atomic swap

improve documentation for the various error types (#5233)

http3: tighten checks for incorrect use of RequestStream (#5231)

The RequestStream is a low-level API that’s used by WebTransport,
CONNECT-UDP and CONNECT-IP. Methods on the RequestStream must be called
in a certain order, and we should detect misuse of the API.

http3: avoid reinitilising the frame parser on the stream (#5232)

No functional change expected.

http3: remove deprecated RoundTripper (#5230)

http3: explicitly expose all method on the RequestStream (#5216)

http3: add ClientConn.Context, CloseWithError and Conn (#5219)

ci: add Go 1.25rc1 to tested Go versions (#5224)

* ci: add Go 1.25rc1 to tested Go versions

* fix rc name

ci: enable the usetesting linter (#5222)

This linter is useful to detect the creation of temporary directories
and the setting of environment variables in tests.

ci: enable the nolintlint linter (#5221)

This linter helps us find unneeded nolint statements.

rename Conn receiver variable (#5215)

No functional change expected.

http3: remove deprecated SingleDestinationRoundTripper type (#5217)

http3: convert Connection interface to Conn struct (#5204)

convert Connection interface to Conn struct (#5195)

explictly expose all method on the Stream (#5214)

This hides the fact that a Stream is composed of a ReceiveStream and a
SendStream. This is an implementation detail and should not be exposed.

handle stream-related frame in the streams map (#5212)

* handle stream-related frame in the streams map

* remove stream manager interface and mock

remove connection flow controller mock (#5213)

No functional change expected.