Use the correct MTU for prev_path

quinn-rs / quinn

Async-friendly QUIC implementation in Rust

Apache License 2.0

3.57k stars 364 forks source link

Closed nemethf closed 2 months ago

nemethf commented 3 months ago

I've just bumped into this. I'm not confident this is really a bug, but using the MTU of a different path seems a wrong thing to do.

djc commented 3 months ago

@nemethf did you want to follow up on this?

nemethf commented 3 months ago

@nemethf did you want to follow up on this?

Should I update the PR with reserve(MIN_INITIAL_SIZE)? (I'm okay with @Ralith or you do it.)

djc commented 3 months ago

@nemethf did you want to follow up on this?

Should I update the PR with reserve(MIN_INITIAL_SIZE)? (I'm okay with @Ralith or you do it.)

Would be nice!

nemethf commented 2 months ago

@nemethf did you want to follow up on this?

I force-pushed a version suggested by @Ralith. Thanks.