feat(quic): added more quic transport config parameters
Description
Allow the users of the QUIC transport to control more QUIC configurations when they provide them.
Notes & open questions
- Defaults change nothing.
- Updated QUINN version and added QUINN-proto
Change checklist
- [x] I have performed a self-review of my own code
- [x] I have made corresponding changes to the documentation
- [ ] I have added tests that prove my fix is effective or that my feature works
- [x] A changelog entry has been made in the appropriate crates
Thanks for the PR!
This adds a lot of new parameters that are all just quinn
TransportConfig. Wdyt of, instead of exposing each parameter separately, we have one functionpub fn with_transport_config<F: Fn(&mut quinn::TransportConfig)>(f: F)where users can modify the config directly?
We'd only need to be careful that there is no breaking change in
quinn::TransportConfigwhen we upgrade the dependency. But I think it's easy to add a test for that.
Discussed with @jxs out-of-band. We decided to not expose quinn::TransportConfig so we don't risk breaking our API when quinn::TransportConfig has a breaking change. So the current approach of this PR is fine.
Where does that leave this PR? what do I need to change for this PR to be approved? I see there is some CHANGELOG issue?