quinn-rs / quinn

Async-friendly QUIC implementation in Rust
Apache License 2.0
3.76k stars 380 forks source link

New release #1352

Closed sakridge closed 2 years ago

sakridge commented 2 years ago

Could there be a new release please with set_max_concurrent_uni_streams?

Ralith commented 2 years ago

Sure, though we should probably roll back #1269 and maybe get async-std support merged first.

sakridge commented 2 years ago

Any updates?

Ralith commented 2 years ago

I've been hoping for feedback on https://github.com/quinn-rs/quinn/pull/1357, but it doesn't seem to be happening. In the mean time I've finished reviewing https://github.com/quinn-rs/quinn/pull/1346, which now needs some additional work.

Do you have a pressing need for a release? A git dependency is usually sufficient for development purposes, but perhaps we could backport the new interfaces you're interested in to 0.8 if a formal release is important and the breaking changes don't get sorted out soon.

sakridge commented 2 years ago

Do you have a pressing need for a release? A git dependency is usually sufficient for development purposes, but perhaps we could backport the new interfaces you're interested in to 0.8 if a formal release is important and the breaking changes don't get sorted out soon.

Yes. We would like to ship code based on this feature. We can't ship based on a git dependency.

djc commented 2 years ago

We're referring to #1315 here, right?

sakridge commented 2 years ago

We're referring to https://github.com/quinn-rs/quinn/pull/1315 here, right?

yes.

Ralith commented 2 years ago

Fixed by (publication of) https://github.com/quinn-rs/quinn/pull/1358.