quinn-rs / quinn

Async-friendly QUIC implementation in Rust
Apache License 2.0
3.57k stars 364 forks source link

docs(quinn): Clarify effects of setting AckFrequencyConfig #1894

Closed gretchenfrage closed 2 weeks ago

gretchenfrage commented 2 weeks ago

Per discussion in Discord [link], the actual extent to which setting ack_frequency_config to None disables the acknowledgement frequency extension is less than the documentation seems to imply. An endpoint with the ack frequency config set to None will still:

The only thing it will not do is send the peer an ACK_FREQUENCY frame.

This PR updates the docs to reflect the behavior more closely.

Ralith commented 2 weeks ago

Thanks!