Merge pull request #41 from jawah/bugfix-large-header

🔖 release 1.2.1
jawah · Oct 15, 2024 · ed2f874 · ed2f874
2 parents fb00a76 + cb08404
commit ed2f874
Show file tree

Hide file tree

Showing 6 changed files with 50 additions and 48 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -40,7 +40,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, macos-13, windows-latest]
+        os: [ubuntu-22.04, macos-13, windows-latest]
         python_version: ['3.7', '3.8', '3.9', '3.10', '3.11', '3.12', '3.13', 'pypy-3.9', 'pypy-3.10']
         exclude:
           # circumvent wierd issue with qh3.asyncio+windows+proactor loop...
@@ -106,7 +106,7 @@ jobs:
         name: Run tests
 
   linux:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     needs:
       - test
       - lint

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -1,3 +1,13 @@
+1.2.1 (2024-10-15)
+====================
+
+**Fixed**
+- Large HTTP headers cannot be encoded to be sent.
+
+**Changed**
+- Upgrade aws-lc-rs to v1.10.0
+- Update rustls to v0.23.14
+
 1.2.0 (2024-09-28)
 ====================
 

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "qh3"
-version = "1.2.0"
+version = "1.2.1"
 edition = "2021"
 rust-version = "1.75"
 license = "BSD-3"

diff --git a/qh3/__init__.py b/qh3/__init__.py
@@ -13,7 +13,7 @@
 from .quic.packet import QuicProtocolVersion
 from .tls import CipherSuite, SessionTicket
 
-__version__ = "1.2.0"
+__version__ = "1.2.1"
 
 __all__ = (
     "connect",

diff --git a/src/headers.rs b/src/headers.rs
@@ -29,7 +29,6 @@ unsafe impl Send for QpackEncoder {}
 
 #[pymethods]
 impl QpackEncoder {
-    // feed_decoder(self, data: bytes) -> None
 
     #[new]
     pub fn py_new() -> Self {
@@ -93,11 +92,10 @@ impl QpackEncoder {
                     )
                 );
             },
-            Err(_) => {
-                return Err(EncoderStreamError::new_err("unable to encode headers"));
+            Err(abc) => {
+                return Err(EncoderStreamError::new_err(format!("unable to encode headers {:?}", abc)));
             }
         }
-
     }
 }