ci: use codecov-action instead of codecov/test-results-action (#5504)

* Initial plan

* Replace deprecated codecov/test-results-action with codecov-action@v5

Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>
---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>

simplify ReceiveStream.Read implementation (#5500)

* simplify check for stream cancellations

* only arm deadline timer if there’s no data to read

* remove redundant nil check

expose SetReliableBoundary method on the Stream (#5498)

This function was already exposed on the SendStream.

remove deprecated ClientHelloInfo (#5497)

ackhandler: only generate RTT sample for the last ack-eliciting packet (#5493)

This logic now works across packet number spaces. It avoids inflated RTT measurements when packets are buffered during the handshake.

ackhandler: record RTT measurements for non-ack-eliciting packets (#5494)

* ackhandler: also save non-ack-eliciting packets in the packet history

* ackhandler: also measure RTTs for non-ack-eliciting packets

utils: make TestAddTimestamp work in all time zones (#5492)

ci: add Go 1.26rc1 to tested Go versions (#5486)

* ci: add Go 1.26rc1 to tested Go versions

* don't run golangci-lint on Go 1.26 yet

update the logo (#5484)

* update the logo

* add a license for the logo

* add a little bit of margin around the PNG logo

* use a more restrictive license

ci: bump actions/upload-artifact from 5 to 6 (#5483)

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

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  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>

ci: bump actions/cache from 4 to 5 (#5482)

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

---
updated-dependencies:
- dependency-name: actions/cache
  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>

improve batch packet processing logic (#5478)

1. More aggressively process packets that arrive during the processing
   of previously packets. This is especially relevant during the
   handshake.
2. Limit batches to 32 packets (previously: 256).

allow packing of ACKs in coalesced packets (#5477)

Sending ACKs in coalesced packets means that we can coalesce more
aggressively, reducing the number of packets sent during the handshake.

This makes use of the recently added AckFrame.Truncate function to
ensure that the ACK frame will always fit.

wire: add a function to trunctate an ACK frame to a given size (#5476)

wire: add a benchmark for ACK frame serialization (#5473)

ackhandler: remove ReceivedPacketHandler interface, use struct directly (#5472)

refactor connection tests to remove ReceivedPacketHandler mock (#5468)

ackhandler: disentangle ReceivedPacketHandler and SentPacketHandler (#5469)

allow processing of multiple packets before handshake completion (#5451)

This is especially valuable:
* on the server side: if the ClientHello spans multiple packets (as it
  now does due to PQ cryptography)
* on the client side: if the server’s certificate span multiple packets
  as they do)

This avoids generating a flood of acknowledgments, since packets in the
Initial and in the Handshake packet number space are acknowledged
immediately.

use synctest for 0-RTT integration tests (#5465)

use synctest for the RTT and reordering tests (#5464)

simnet: move latency configuration to Router, simplify queueing logic (#5463)

* simnet: update to new CoDel queueing logic

This includes all changes up to
https://github.com/MarcoPolo/simnet/commit/b41360a50f1115e66d3ead762fd679faba0cc1dd.

* simnet: remove bandwidth link setting, simplify queueing logic

fix flaky TestConnectionCongestionControl (#5467)

* Initial plan

* Fix flaky TestConnectionCongestionControl by adding missing synctest.Wait()

Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>
---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>

handshake: fix QUIC events with session tickets disabled on Go 1.26 (#5462)

use synctest in the session resumption test (#5424)

handshake: add a benchmark for token decoding (#5460)

handshake: pass cipherSuite by value instead of pointer (#5459)

ci: bump actions/checkout from 5 to 6 (#5458)

Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [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/v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  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>

qlog the datagram_id for long header and coalesced packets (#5455)

* simplify logging of short header packets

No functional change expected.

* qlog: add datagram_id field to packet evens

* qlog the datagram_id for long header and coalesced packets

No datagram_id is logged for short header packets.

qlog: add a DebugEvent (#5456)

http3: limit size of decompressed headers (#5452)

qlogwriter: use synctest to make tests deterministic (#5454)

README: add nodepass to list of projects (#5448)

* README: add nodepass to list of projects

* README: reorder nodepass in list of projects

don’t arm connection timer for connection ID retirement (#5449)

It should be sufficient to retire connection IDs the next time the run
loops wakes up anyway, e.g. when a packet is received, or when a new
packet is sent.

ackhandler: don’t generate an immediate ACK for the first packet (#5447)

Instead apply the regular RFC 9000 logic and send an acknowledgment
after receiving two ack-eliciting packets.

add documentation for Conn.NextConnection (#5442)

http3: use QPACK_DECOMPRESSION_FAILED for QPACK errors (#5439)

* update testify to v1.11.1

* http3: use QPACK_DECOMPRESSION_FAILED for QPACK errors

update qpack to v0.6.0 (#5434)

* update qpack to v0.6.0

* qlog invalid trailers

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

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

http3: add a benchmark for header parsing (#5435)

http3: make Transport.MaxResponseBytes an int (#5433)

Both http.Server.MaxHeaderBytes and http3.Server.MaxHeaderBytes are ints
as well, so this is more consistent.

http3: read response after encountering error sending the request (#5432)

If the error is a connection-wide error, or if the receive side of the
stream was reset, reading the response will fail immediately. However,
if only the send side was reset, this enables the client to read the
server’s response (e.g. a HTTP status code).

http3: send SETTINGS_MAX_FIELD_SECTION_SIZE in the SETTINGS frame (#5431)

* http3/qlog: implement qlogging of SETTINGS_MAX_FIELD_SECTION_SIZE

* http3: send SETTINGS_MAX_FIELD_SECTION_SIZE in the SETTINGS frame

qlogwriter: fix storing of event schemas (#5430)

HTTP/3 qlogging (and qlogging for any other application protocol) only
works if the event schemas are stored.

ci: bump golangci/golangci-lint-action from 8 to 9 (#5426)

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 8 to 9.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](https://github.com/golangci/golangci-lint-action/compare/v8...v9)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-version: '9'
  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>

qlog: split the PTO count updates ouf of the MetricsUpdated event (#5421)

* qlog: split the PTO count updates ouf of the MetricsUpdated event

This allows us to not use pointers for all member variables of both
events.

* ackhandler: avoid spurious call to GetCongestionWindow

qlog: rework the ConnectionClosed event (#5417)

ackhandler: fix qlogging of RTT values (#5418)

use synctest for the stateless reset tests (#5415)

qlog: rename owner to initiator (#5416)

Owner was renamed to Initiator in draft-ietf-qlog-quic-events-11.

use synctest in handshake RTT tests (#5414)

ci: update golangci-lint to v2.6.0 (#5412)

* ci: update golangci-lint to v2.6.0

* supress synctest linter warning

ci: bump actions/upload-artifact from 4 to 5 (#5411)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  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>

quicvarint: improve panic message for numbers larger 2^62 (#5410)

ci: use gcassert to check that quicvarint.Len is inlined (#5409)

fix flaky TestServerTransportClose (#5407)

* Initial plan

* Fix TestServerTransportClose race condition by accepting conn1 before dialing conn2

Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>
---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>

fix flaky TestConnectionUnpackFailureDropped (#5382)

* Initial plan

* Fix flaky TestConnectionUnpackFailureDropped by adding synctest.Wait

Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>
* formatting

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: marten-seemann <1478487+marten-seemann@users.noreply.github.com>
Co-authored-by: Marten Seemann <martenseemann@gmail.com>

use synctest for the timeout tests (#5403)

use synctest for the datagram test (#5398)

use synctest for the handshake drop test (#5397)

This test used to take 15-30s locally, and even more on CI. It now runs in less than 1ms.

use synctest for the packet drop test (#5393)

use simnet in CONNECTION_CLOSE retransmission test (#5395)

* use simnet in CONNECTION_CLOSE retransmission test

* fix race

use synctest for transport tests (#5391)

* use synctest for transport tests

* disable TestTransportReplaceWithClosed on Go 1.24

add simnet package to simulate a net.PacketConn in memory (#5385)

* Implement simnet

* simnet: remove separate license

* simnet: remove go.mod, use standard require package

* simnet: add README

* simnet: use synctest wrapper in tests

* simnet: minor code cleanup

* simnet: expose Packet.Data

* simnet: explose Simnet.Router

* simnet: remove SimpleFirewallRouter

* simnet: remove stray fmt.Println in tests

* fix deadline check for write deadlines

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* simnet: fix SetReadDeadline logic

---------

Co-authored-by: Marco Munizaga <git@marcopolo.io>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

congestion: avoid overflows when calculating pacer budget (#5390)

* congestion: prevent uint64 overflow in pacer

This should never happen for bandwidths even remotely possible in
real-world scenarios, but it’s better to be safe.

* congestion: add a benchmark test for the pacer

* add a comment

use default RTT (100ms) for 0-RTT if no prior estimate (#5388)

* utils: initialize the {Smoothed, Latest, Min}RTT to 100ms

* utils: use time.Duration.Nanoseconds instead of uint64

No functional change expected.

* congestion: better check to avoid division by zero

http3: add the qlog event schema to trace header (#5383)

http3: qlog reserved, unsupported and unknown frames (#5381)

http3: qlog the frame length and payload length of parsed frames (#5380)

http3: qlog sent and received SETTINGS frames (#5379)

* http3: implement qlog serialization for SETTINGS frames

* http3: qlog sent and received SETTINGS frames

http3: move qlogging of frames into the frame parser (#5378)

http3: qlog sent and received DATAGRAMs (#5375)

http3: qlog sent and received GOAWAY frames (#5376)

README: update OSS-Fuzz link (#5377)

oss-fuzz issues has been migrated to issues.oss-fuzz.com and the project field is not searcheable.

http3: add basic client-side qlog support (#5374)

http3: fix qlog encoding of frame_parsed and frame_created events (#5372)

qlog: use PathEndpointInfo in connection_started (#5368)

* qlog: use PathEndpointInfo in connection_started

* correctly deal with dual-stack addresses

jsontext: add support for encoding null (#5371)

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>