chore: Bump qlog to 0.13 and deal with the fallout

Fixes #1826
mozilla · Apr 15, 2024 · 02c69cf · 02c69cf
1 parent 34d9603
commit 02c69cf
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 8 deletions.
diff --git a/neqo-bin/Cargo.toml b/neqo-bin/Cargo.toml
@@ -34,7 +34,7 @@ neqo-crypto = { path = "./../neqo-crypto" }
 neqo-http3 = { path = "./../neqo-http3" }
 neqo-qpack = { path = "./../neqo-qpack" }
 neqo-transport = { path = "./../neqo-transport" }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 quinn-udp = { git = "https://github.com/quinn-rs/quinn/", rev = "a947962131aba8a6521253d03cc948b20098a2d6" }
 regex = { version = "1.9", default-features = false, features = ["unicode-perl"] }
 tokio = { version = "1", default-features = false, features = ["net", "time", "macros", "rt", "rt-multi-thread"] }

diff --git a/neqo-common/Cargo.toml b/neqo-common/Cargo.toml
@@ -17,7 +17,7 @@ workspace = true
 enum-map = { version = "2.7", default-features = false }
 env_logger = { version = "0.10", default-features = false }
 log = { version = "0.4", default-features = false }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 time = { version = "0.3", default-features = false, features = ["formatting"] }
 
 [dev-dependencies]

diff --git a/neqo-http3/Cargo.toml b/neqo-http3/Cargo.toml
@@ -19,7 +19,7 @@ neqo-common = { path = "./../neqo-common" }
 neqo-crypto = { path = "./../neqo-crypto" }
 neqo-qpack = { path = "./../neqo-qpack" }
 neqo-transport = { path = "./../neqo-transport" }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 sfv = { version = "0.9", default-features = false }
 smallvec = { version = "1.11", default-features = false }
 url = { version = "2.5", default-features = false }

diff --git a/neqo-qpack/Cargo.toml b/neqo-qpack/Cargo.toml
@@ -17,7 +17,7 @@ log = { version = "0.4", default-features = false }
 neqo-common = { path = "./../neqo-common" }
 neqo-crypto = { path = "./../neqo-crypto" }
 neqo-transport = { path = "./../neqo-transport" }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 static_assertions = { version = "1.1", default-features = false }
 
 [dev-dependencies]

diff --git a/neqo-transport/Cargo.toml b/neqo-transport/Cargo.toml
@@ -17,7 +17,7 @@ indexmap = { version = "1.9", default-features = false }
 log = { version = "0.4", default-features = false }
 neqo-common = { path = "../neqo-common" }
 neqo-crypto = { path = "../neqo-crypto" }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 smallvec = { version = "1.11", default-features = false }
 
 [dev-dependencies]

diff --git a/neqo-transport/src/qlog.rs b/neqo-transport/src/qlog.rs
@@ -392,8 +392,14 @@ impl From<&Frame<'_>> for QuicFrame {
         match frame {
             // TODO: Add payload length to `QuicFrame::Padding` once
             // https://github.com/cloudflare/quiche/pull/1745 is available via the qlog crate.
-            Frame::Padding { .. } => QuicFrame::Padding,
-            Frame::Ping => QuicFrame::Ping,
+            Frame::Padding(len) => QuicFrame::Padding {
+                length: None,
+                payload_length: u32::from(*len),
+            },
+            Frame::Ping => QuicFrame::Ping {
+                length: None,
+                payload_length: None,
+            },
             Frame::Ack {
                 largest_acknowledged,
                 ack_delay,
@@ -418,6 +424,8 @@ impl From<&Frame<'_>> for QuicFrame {
                     ect1: None,
                     ect0: None,
                     ce: None,
+                    length: None,
+                    payload_length: None,
                 }
             }
             Frame::ResetStream {
@@ -428,13 +436,17 @@ impl From<&Frame<'_>> for QuicFrame {
                 stream_id: stream_id.as_u64(),
                 error_code: *application_error_code,
                 final_size: *final_size,
+                length: None,
+                payload_length: None,
             },
             Frame::StopSending {
                 stream_id,
                 application_error_code,
             } => QuicFrame::StopSending {
                 stream_id: stream_id.as_u64(),
                 error_code: *application_error_code,
+                length: None,
+                payload_length: None,
             },
             Frame::Crypto { offset, data } => QuicFrame::Crypto {
                 offset: *offset,

diff --git a/test-fixture/Cargo.toml b/test-fixture/Cargo.toml
@@ -19,7 +19,7 @@ neqo-crypto = { path = "../neqo-crypto" }
 neqo-http3 = { path = "../neqo-http3" }
 neqo-qpack = { path = "../neqo-qpack" }
 neqo-transport = { path = "../neqo-transport" }
-qlog = { version = "0.12", default-features = false }
+qlog = { version = "0.13", default-features = false }
 
 [features]
 bench = []