http3: Pass original Conn to ConnContext

[rthellend]

Server.ConnContext should receive the original quic.Connection instead of an internal wrapper.

#4479

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.90%. Comparing base (3122ca0) to head (f5cd16a).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4480      +/-   ##
==========================================
- Coverage   84.92%   84.90%   -0.02%     
==========================================
  Files         154      154              
  Lines       14676    14676              
==========================================
- Hits        12463    12460       -3     
- Misses       1700     1702       +2     
- Partials      513      514       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[marten-seemann]

Should we have a test for this? I don't want to break this again in the future.

[rthellend]

Should we have a test for this? I don't want to break this again in the future.

I updated the test in http3/server_test.go

[marten-seemann]

Thanks!

I saw you downgraded back to v0.42. Would a v0.43.1 release be helpful for your project, once this PR is merged?

[rthellend]

Thanks!

I saw you downgraded back to v0.42. Would a v0.43.1 release be helpful for your project, once this PR is merged?

I would certainly use v0.43.1 if you release it, yes. Thanks!

[marten-seemann]

@rthellend v0.43.1 is here: https://github.com/quic-go/quic-go/releases/tag/v0.43.1

[rthellend]

Thanks!