diff --git a/neqo-client/src/main.rs b/neqo-client/src/main.rs index de7da48a27..e204b179cc 100644 --- a/neqo-client/src/main.rs +++ b/neqo-client/src/main.rs @@ -333,7 +333,7 @@ impl QuicParameters { params.versions(first.0, all.iter().map(|&x| x.0).collect()) } else { let version = match alpn { - "h3" | "hq-interop" => Version::default(), + "h3" | "hq-interop" => Version::Version1, "h3-29" | "hq-29" => Version::Draft29, "h3-30" | "hq-30" => Version::Draft30, "h3-31" | "hq-31" => Version::Draft31, diff --git a/neqo-interop/src/main.rs b/neqo-interop/src/main.rs index ef5646ea73..c274c57d0f 100644 --- a/neqo-interop/src/main.rs +++ b/neqo-interop/src/main.rs @@ -736,11 +736,9 @@ fn run_test<'t>(peer: &Peer, test: &'t Test) -> (&'t Test, String) { return (test, String::from("OK")); } Test::H9 => test_h9(&nctx, &mut client), - Test::H3 => test_h3(&nctx, peer, client, test), + Test::H3 | Test::D => test_h3(&nctx, peer, client, test), Test::VN => unimplemented!(), - Test::R => test_h3_rz(&nctx, peer, client, test), - Test::Z => test_h3_rz(&nctx, peer, client, test), - Test::D => test_h3(&nctx, peer, client, test), + Test::R | Test::Z => test_h3_rz(&nctx, peer, client, test), }; if let Err(e) = res { diff --git a/neqo-transport/src/connection/mod.rs b/neqo-transport/src/connection/mod.rs index 270b96f08f..e8d10387e3 100644 --- a/neqo-transport/src/connection/mod.rs +++ b/neqo-transport/src/connection/mod.rs @@ -1019,7 +1019,7 @@ impl Connection { let res = self.client_start(now); self.absorb_error(now, res); } - (State::Init, Role::Server) | (State::WaitInitial, Role::Server) => { + (State::Init | State::WaitInitial, Role::Server) => { return Output::None; } _ => { @@ -1292,8 +1292,7 @@ impl Connection { self.handle_retry(packet, now); return Ok(PreprocessResult::Next); } - (PacketType::Handshake, State::WaitInitial, Role::Client) - | (PacketType::Short, State::WaitInitial, Role::Client) => { + (PacketType::Handshake | PacketType::Short, State::WaitInitial, Role::Client) => { // This packet can't be processed now, but it could be a sign // that Initial packets were lost. // Resend Initial CRYPTO frames immediately a few times just @@ -1306,9 +1305,7 @@ impl Connection { self.crypto.resend_unacked(PacketNumberSpace::Initial); } } - (PacketType::VersionNegotiation, ..) - | (PacketType::Retry, ..) - | (PacketType::OtherVersion, ..) => { + (PacketType::VersionNegotiation | PacketType::Retry | PacketType::OtherVersion, ..) => { self.stats .borrow_mut() .pkt_dropped(format!("{:?}", packet.packet_type())); diff --git a/neqo-transport/src/qlog.rs b/neqo-transport/src/qlog.rs index 021367d8da..59168147d2 100644 --- a/neqo-transport/src/qlog.rs +++ b/neqo-transport/src/qlog.rs @@ -119,8 +119,7 @@ pub fn connection_state_updated(qlog: &mut NeqoQlog, new: &State) { let ev_data = EventData::ConnectionStateUpdated(ConnectionStateUpdated { old: None, new: match new { - State::Init => ConnectionState::Attempted, - State::WaitInitial => ConnectionState::Attempted, + State::Init | State::WaitInitial => ConnectionState::Attempted, State::WaitVersion | State::Handshaking => ConnectionState::HandshakeStarted, State::Connected => ConnectionState::HandshakeCompleted, State::Confirmed => ConnectionState::HandshakeConfirmed,