Marten Seemann

Results 924 comments of Marten Seemann

It’s just the ping cross version test plan, modified such that I can record a qlog. That’s all the changes I made.

I rebased this PR and removed the commit that removes the `mocknet` package.

Merging this PR, with the understanding that this will get refactored very soon.

And here with a race condition: ``` === RUN TestIDService 2022-11-11T07:40:51.153Z DEBUG net/identify identify/id.go:412 /ipfs/id/1.0.0 sent message to 12D3KooWDD9fga7C4XfNWC5RbNtBBmV4dYBHGKkj8KxUpTSud6jN /ip4/127.0.0.1/udp/57628/quic 2022-11-11T07:40:51.156Z DEBUG net/identify identify/id.go:444 /ipfs/id/1.0.0 received message from 12D3KooWKwAUcxwuwbwAe7nwqHA39AcwiEPASzQqsqx2sHWAJP3z /ip4/127.0.0.1/udp/38869/quic...

Implementing that draft would be a massive undertaking that would only make sense with a particular application protocol in mind, and a very sophisticated benchmarking suite.

@tie What's the status of this PR? Are you still working on it?

@mojatter Can you please sign the CLA (see https://github.com/lucas-clemente/quic-go/pull/3465#issuecomment-1170788494). We'd need that so we can merge the PR.

Why can't we make this analogous to the H2 client?

Yes, something along those lines. A few thoughts: 1. It probably makes sense to only do this for Initial packets. 2. It should use the CONNECTION_REFUSED error. The tricky thing...